X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Ffigured-bass-engraver.cc;h=32ad86a9eb7592fa42c428ee857f4a6d8399802c;hb=d15c79e957e0cc927996ab3ba3526ef63a323b92;hp=2c3ad0e9e13f7e32aff384466e6f29734a95bfb5;hpb=358d6e99ada8019268ade9ba9080b2d0d88eeb7a;p=lilypond.git

diff --git a/lily/figured-bass-engraver.cc b/lily/figured-bass-engraver.cc
index 2c3ad0e9e1..32ad86a9eb 100644
--- a/lily/figured-bass-engraver.cc
+++ b/lily/figured-bass-engraver.cc
@@ -3,7 +3,7 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 2005--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  (c) 2005--2007 Han-Wen Nienhuys <hanwen@xs4all.nl>
 
 */
 
@@ -57,8 +57,8 @@ struct Figure_group
 
 struct Figured_bass_engraver : public Engraver
 {
-  TRANSLATOR_DECLARATIONS(Figured_bass_engraver);
-  void clear_spanners();
+  TRANSLATOR_DECLARATIONS (Figured_bass_engraver);
+  void clear_spanners ();
   void add_brackets ();
   void create_grobs ();
 
@@ -142,7 +142,12 @@ Figured_bass_engraver::listen_rest (Stream_event *ev)
   if (to_boolean (get_property ("ignoreFiguredBassRest")))
     {
       new_event_found_ = true;
-      ASSIGN_EVENT_ONCE (rest_event_, ev);
+
+      /*
+	No ASSIGN_EVENT_ONCE () ; otherwise we get warnings about
+	polyphonic rests.
+       */
+      rest_event_ = ev;
     }
 }
 
@@ -151,7 +156,7 @@ void
 Figured_bass_engraver::listen_bass_figure (Stream_event *ev)
 {
   new_event_found_ = true;
-  Moment stop  = now_mom () + get_event_length (ev);
+  Moment stop  = now_mom () + get_event_length (ev, now_mom ());
   stop_moment_ = max (stop_moment_, stop);
 
   if (to_boolean (get_property ("useBassFigureExtenders")))
@@ -179,13 +184,13 @@ Figured_bass_engraver::center_continuations (vector<Spanner*> const &consecutive
   if (consecutive_lines.size () == 2)
     {
       vector<Grob*> left_figs;
-      for (vsize j = consecutive_lines.size(); j--;)
+      for (vsize j = consecutive_lines.size (); j--;)
 	left_figs.push_back (consecutive_lines[j]->get_bound (LEFT));
 
       SCM  ga = Grob_array::make_array ();
       unsmob_grob_array (ga)->set_array (left_figs);
 
-      for (vsize j = consecutive_lines.size(); j--;)
+      for (vsize j = consecutive_lines.size (); j--;)
 	consecutive_lines[j]->set_object ("figures",
 					  unsmob_grob_array (ga)->smobbed_copy ());
     }
@@ -195,14 +200,14 @@ void
 Figured_bass_engraver::center_repeated_continuations ()
 {  
   vector<Spanner*> consecutive_lines;
-  for (vsize i = 0; i <= groups_.size(); i++)
+  for (vsize i = 0; i <= groups_.size (); i++)
     {
       if (i < groups_.size ()
 	  && groups_[i].continuation_line_
 	  && (consecutive_lines.empty ()
-	      || (consecutive_lines[0]->get_bound(LEFT)->get_column ()
+	      || (consecutive_lines[0]->get_bound (LEFT)->get_column ()
 	          == groups_[i].continuation_line_->get_bound (LEFT)->get_column ()
-		  && consecutive_lines[0]->get_bound(RIGHT)->get_column ()
+		  && consecutive_lines[0]->get_bound (RIGHT)->get_column ()
 	          == groups_[i].continuation_line_->get_bound (RIGHT)->get_column ())))
 	consecutive_lines.push_back (groups_[i].continuation_line_);	  
       else 
@@ -226,7 +231,7 @@ Figured_bass_engraver::clear_spanners ()
     }
 
   if (to_boolean (get_property ("figuredBassCenterContinuations")))
-    center_repeated_continuations();
+    center_repeated_continuations ();
   
   for (vsize i = 0; i < groups_.size (); i++)
     {
@@ -281,7 +286,7 @@ Figured_bass_engraver::add_brackets ()
 void
 Figured_bass_engraver::process_music ()
 {
-  if (!to_boolean (get_property ("useBassFigureExtenders")))
+  if (alignment_ && !to_boolean (get_property ("useBassFigureExtenders")))
     clear_spanners ();
         
   if (rest_event_)
@@ -349,7 +354,7 @@ Figured_bass_engraver::process_music ()
   if (use_extenders)
     {
       vector<int> junk_continuations;
-      for (vsize i = 0; i < groups_.size(); i++)
+      for (vsize i = 0; i < groups_.size (); i++)
 	{
 	  Figure_group &group = groups_[i];
 
@@ -384,7 +389,7 @@ Figured_bass_engraver::process_music ()
 	{
 	  for (vsize i = 0; i <= junk_continuations.size (); i++)
 	    {
-	      if (i < junk_continuations.size()
+	      if (i < junk_continuations.size ()
 		  && (i == 0 || junk_continuations[i-1] == junk_continuations[i] - 1))
 		consecutive.push_back (groups_[junk_continuations[i]].continuation_line_);
 	      else 
@@ -416,7 +421,7 @@ Figured_bass_engraver::create_grobs ()
   alignment_->set_bound (RIGHT, muscol);
 
   SCM proc = get_property ("figuredBassFormatter");
-  for (vsize i = 0; i < groups_.size(); i++)
+  for (vsize i = 0; i < groups_.size (); i++)
     {
       Figure_group &group = groups_[i];
       
@@ -477,15 +482,14 @@ Figured_bass_engraver::create_grobs ()
 
 ADD_TRANSLATOR (Figured_bass_engraver,
 		/* doc */
-
 		"Make figured bass numbers.",
+
 		/* create */
 		"BassFigure "
 		"BassFigureAlignment "
 		"BassFigureBracket "
 		"BassFigureContinuation "
-		"BassFigureLine "
-		,
+		"BassFigureLine ",
 
 		/* read */
 		"figuredBassAlterationDirection "
@@ -493,8 +497,8 @@ ADD_TRANSLATOR (Figured_bass_engraver,
 		"figuredBassFormatter "
 		"implicitBassFigures "
 		"useBassFigureExtenders "
-		"ignoreFiguredBassRest "
-		,
+		"ignoreFiguredBassRest ",
 
 		/* write */
-		"");
+		""
+		);