X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Ftiming-translator.cc;h=d7929181716a635177523b7c657afd935dd33d05;hb=f354ae4e1a6dab6ad4b1a98c2d732f542a05a02e;hp=85bffc00b68b252df5a0b06522e5df954c936423;hpb=337f6bac894a741641eb9a0d78b3bc2c376fad12;p=lilypond.git

diff --git a/lily/timing-translator.cc b/lily/timing-translator.cc
index 85bffc00b6..d792918171 100644
--- a/lily/timing-translator.cc
+++ b/lily/timing-translator.cc
@@ -4,39 +4,32 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 1997--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1997--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
 */
 
-#include "warn.hh"
 #include "timing-translator.hh"
 
+#include "warn.hh"
 #include "translator-group.hh"
 #include "global-context.hh"
 #include "multi-measure-rest.hh"
 
-
 void
 Timing_translator::stop_translation_timestep ()
 {
   Global_context *global = get_global_context ();
 
-  /* allbars == ! skipbars */
-  SCM sb = get_property ("skipBars");
-  bool allbars = !to_boolean (sb);
-
-  // urg: multi bar rests: should always process whole of first bar?
-  SCM tim = get_property ("timing");
-  bool timb = to_boolean (tim);
-  if (timb && allbars)
+  if (to_boolean (get_property ("timing"))
+      && !to_boolean (get_property ("skipBars")))
     {
-      Moment barleft = (measure_length () - measure_position ());
+      Moment barleft = (measure_length () - measure_position (context ()));
       Moment now = now_mom ();
 
       if (barleft > Moment (0)
 	  /*
 	    Hmm. We insert the bar moment every time we process a
 	    moment.  A waste of cpu?
-	   */
+	  */
 	  && !now.grace_part_)
 	global->add_moment_to_process (now + barleft);
     }
@@ -45,19 +38,20 @@ Timing_translator::stop_translation_timestep ()
 void
 Timing_translator::initialize ()
 {
+  context ()->add_alias (ly_symbol2scm ("Timing"));
+  context ()->set_property ("currentBarNumber", scm_from_int (1));
+  context ()->set_property ("internalBarNumber", scm_from_int (1));
 
+  context ()->set_property ("timeSignatureFraction",
+			    scm_cons (scm_from_int (4), scm_from_int (4)));
   /*
-    move this to engraver-init.ly? 
-   */
- 
-  daddy_context_->set_property ("timing" , SCM_BOOL_T);  
-  daddy_context_->set_property ("currentBarNumber" , gh_int2scm (1));
-
-  daddy_context_->set_property ("timeSignatureFraction",
-				gh_cons (gh_int2scm (4), gh_int2scm (4)));
-  daddy_context_->set_property ("measurePosition", Moment (Rational (0)).smobbed_copy ());
-  daddy_context_->set_property ("measureLength", Moment (Rational (1)).smobbed_copy ());
-  daddy_context_->set_property ("beatLength", Moment (Rational (1,4)).smobbed_copy ());
+    Do not init measurePosition; this should be done from global
+    context.
+  */
+  context ()->set_property ("measureLength",
+			    Moment (Rational (1)).smobbed_copy ());
+  context ()->set_property ("beatLength",
+			    Moment (Rational (1, 4)).smobbed_copy ());
 }
 
 Rational
@@ -72,43 +66,26 @@ Timing_translator::measure_length () const
 
 Timing_translator::Timing_translator ()
 {
-
-}
-
-Moment
-Timing_translator::measure_position () const
-{
-  SCM sm = get_property ("measurePosition");
-  
-  Moment m   =0;
-  if (unsmob_moment (sm))
-    {
-      m = *unsmob_moment (sm);
-      while (m.main_part_ < Rational (0))
-	m.main_part_ += measure_length ();
-    }
-  
-  return m;
 }
 
 void
 Timing_translator::start_translation_timestep ()
 {
-  Global_context *global =get_global_context ();
+  Global_context *global = get_global_context ();
 
   Moment now = global->now_mom ();
-  Moment dt = now  - global->previous_moment ();
+  Moment dt = now - global->previous_moment ();
   if (dt < Moment (0))
     {
-      programming_error ("Moving backwards in time");
+      programming_error ("moving backwards in time");
       dt = 0;
     }
   else if (dt.main_part_.is_infinity ())
     {
-      programming_error ("Moving infinitely to future");
+      programming_error ("moving infinitely to future");
       dt = 0;
     }
-  
+
   if (!dt.to_bool ())
     return;
 
@@ -116,36 +93,54 @@ Timing_translator::start_translation_timestep ()
 
   SCM s = get_property ("measurePosition");
   if (unsmob_moment (s))
-    {
-      measposp = *unsmob_moment (s);
-    }
+    measposp = *unsmob_moment (s);
   else
     {
       measposp = now;
-      daddy_context_->set_property ("measurePosition", measposp.smobbed_copy ());
+      context ()->set_property ("measurePosition",
+				measposp.smobbed_copy ());
     }
-  
+
   measposp += dt;
-  
-  SCM barn = get_property ("currentBarNumber");
-  int b = 0;
-  if (gh_number_p (barn))
-    {
-      b = gh_scm2int (barn);
-    }
+
+  int current_barnumber = robust_scm2int (get_property ("currentBarNumber"), 0);
+  int internal_barnumber = robust_scm2int (get_property ("internalBarNumber"), 0);
 
   SCM cad = get_property ("timing");
-  bool c= to_boolean (cad);
+  bool c = to_boolean (cad);
 
   Rational len = measure_length ();
   while (c && measposp.main_part_ >= len)
     {
       measposp.main_part_ -= len;
-      b ++;
+      current_barnumber ++;
+      internal_barnumber ++;
     }
 
-  daddy_context_->set_property ("currentBarNumber", gh_int2scm (b));
-  daddy_context_->set_property ("measurePosition", measposp.smobbed_copy ());
+  context ()->set_property ("currentBarNumber", scm_from_int (current_barnumber));
+  context ()->set_property ("internalBarNumber", scm_from_int (internal_barnumber));
+  context ()->set_property ("measurePosition", measposp.smobbed_copy ());
 }
 
-ENTER_DESCRIPTION (Timing_translator,"","","","","","");
+#include "translator.icc"
+
+ADD_TRANSLATOR (Timing_translator,
+		"This engraver adds the alias "
+		"@code{Timing} to its containing context."
+		"Responsible for synchronizing timing information from staves.  "
+		"Normally in @code{Score}.  In order to create polyrhythmic music, "
+		"this engraver should be removed from @code{Score} and placed in "
+		"@code{Staff}. "
+		"\n\nThis engraver adds the alias @code{Timing} to its containing context.",
+
+		"", "",
+
+		"internalBarNumber "
+		"currentBarNumber "
+		"measureLength "
+		"measurePosition ",
+
+		"internalBarNumber "
+		"currentBarNumber "
+		"measurePosition "
+		);