X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Finstrument-name-engraver.cc;h=d35e065afb7459bf691acec9a7fcf4a9d0e4bc0a;hb=6a33b5cb60d40f9ae549deac8d0e6ebebf17af60;hp=2f361f0fe5204f0d2a3393973d733fb0b3043ebd;hpb=01da54f59a4e91149bdce7b1f63f3cbfacca5bd1;p=lilypond.git

diff --git a/lily/instrument-name-engraver.cc b/lily/instrument-name-engraver.cc
index 2f361f0fe5..d35e065afb 100644
--- a/lily/instrument-name-engraver.cc
+++ b/lily/instrument-name-engraver.cc
@@ -3,11 +3,10 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 2000--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  (c) 2000--2007 Han-Wen Nienhuys <hanwen@xs4all.nl>
 */
 
 #include "engraver.hh"
-#include "spanner.hh"
 #include "pointer-group-interface.hh"
 #include "side-position-interface.hh"
 #include "axis-group-interface.hh"
@@ -25,54 +24,97 @@ public:
 protected:
   Spanner *text_spanner_;
 
+  SCM long_text_;
+  SCM short_text_;
+
+  vector<Grob*> axis_groups_;
+  
   virtual void finalize ();
   DECLARE_ACKNOWLEDGER (axis_group);
   void process_music ();
+  void start_spanner ();
+  void consider_start_spanner ();
+  void stop_spanner ();
 };
 
 Instrument_name_engraver::Instrument_name_engraver ()
 {
   text_spanner_ = 0;
+
+  long_text_ = SCM_EOL;
+  short_text_ = SCM_EOL;
 }
 
 void
 Instrument_name_engraver::process_music ()
 {
-  if (!text_spanner_)
+  consider_start_spanner ();
+}
+
+void
+Instrument_name_engraver::consider_start_spanner ()
+{
+  SCM long_text = get_property ("instrumentName");
+  SCM short_text = get_property ("shortInstrumentName");
+
+  if (!(Text_interface::is_markup (long_text)
+	|| Text_interface::is_markup (short_text)))
     {
-      SCM long_text = get_property ("instrument");
-      SCM short_text = get_property ("instr");
-
-      if (!(Text_interface::is_markup (long_text)
-	    || Text_interface::is_markup (short_text)))
-	{
-	  long_text = get_property ("vocalName");
-	  short_text = get_property ("vocNam");
-	}
+      long_text = get_property ("vocalName");
+      short_text = get_property ("shortVocalName");
+    }
   
-      if (Text_interface::is_markup (long_text)
-	  || Text_interface::is_markup (short_text))
-	{
-	  text_spanner_ = make_spanner ("InstrumentName", SCM_EOL);
-	  
-	  Grob *col = unsmob_grob (get_property ("currentCommandColumn"));
-	  text_spanner_->set_bound (LEFT, col);
-	  text_spanner_->set_property ("text", short_text);
-	  text_spanner_->set_property ("long-text", long_text);
-	}
+  if ((Text_interface::is_markup (long_text)
+       || Text_interface::is_markup (short_text))
+      && (!text_spanner_
+	  || short_text_ != short_text
+	  || long_text_ != long_text))
+    {
+      if (text_spanner_)
+	stop_spanner ();
+
+      
+      short_text_ = short_text;
+      long_text_ = long_text;
+
+      start_spanner ();
     }
 }
 
+void
+Instrument_name_engraver::start_spanner ()
+{
+  text_spanner_ = make_spanner ("InstrumentName", SCM_EOL);
+	  
+  Grob *col = unsmob_grob (get_property ("currentCommandColumn"));
+  text_spanner_->set_bound (LEFT, col);
+  text_spanner_->set_property ("text", short_text_);
+  text_spanner_->set_property ("long-text", long_text_);
+
+  /*
+    UGH, should handle this in Score_engraver.
+  */
+  Grob *system = unsmob_grob (get_property ("rootSystem"));
+  if (system)
+    Axis_group_interface::add_element (system, text_spanner_);
+  else
+    text_spanner_->programming_error ("cannot find root system");
+}
+
+
 void
 Instrument_name_engraver::acknowledge_axis_group (Grob_info info)
 {
-  if (text_spanner_ 
-      && dynamic_cast<Spanner *> (info.grob ())
+  if (dynamic_cast<Spanner *> (info.grob ())
       && Axis_group_interface::has_axis (info.grob (), Y_AXIS)
+
+      /* ugh. */
+
+      && !info.grob ()->internal_has_interface (ly_symbol2scm ("dynamic-interface"))
+      && !info.grob ()->internal_has_interface (ly_symbol2scm ("piano-pedal-interface"))
       && (!Align_interface::has_interface (info.grob ())))
     {
-      Grob *staff = info.grob();
-      Pointer_group_interface::add_grob (text_spanner_, ly_symbol2scm ("elements"), staff);
+      axis_groups_.push_back (info.grob ());
     }
 }
 
@@ -81,21 +123,22 @@ Instrument_name_engraver::finalize ()
 {
   if (text_spanner_)
     {
-      text_spanner_->set_bound (RIGHT,
-				unsmob_grob (get_property ("currentCommandColumn")));
+      stop_spanner ();
+    }
+}
 
-      Pointer_group_interface::set_ordered (text_spanner_, ly_symbol2scm ("elements"), false);
+void
+Instrument_name_engraver::stop_spanner ()
+{
+  for (vsize i = 0; i < axis_groups_.size (); i++)
+    Pointer_group_interface::add_grob (text_spanner_, ly_symbol2scm ("elements"), axis_groups_[i]);
+  
+  text_spanner_->set_bound (RIGHT,
+			    unsmob_grob (get_property ("currentCommandColumn")));
 
-      System *system = get_root_system (text_spanner_);
+  Pointer_group_interface::set_ordered (text_spanner_, ly_symbol2scm ("elements"), false);
 
-      /*
-	UGH, should handle this in Score_engraver.
-       */
-      if (system)
-	Axis_group_interface::add_element (system, text_spanner_);
-      else
-	text_spanner_->programming_error ("can't find root system");
-    }
+  text_spanner_ = 0;
 }
 
 #include "translator.icc"
@@ -103,18 +146,19 @@ Instrument_name_engraver::finalize ()
 ADD_ACKNOWLEDGER (Instrument_name_engraver, axis_group);
 
 ADD_TRANSLATOR (Instrument_name_engraver,
-
 		/* doc */
-		"Creates a system start text for instrument or vocal names.",
+		"Create a system start text for instrument or vocal names.",
 		
 		/* create */
 		"InstrumentName ",
 		
-		/* accept */
-		"",
-		
 		/* read */
-		"vocNam vocalName instrument instr "
-		"currentCommandColumn",
-
-		/* write */ "");
+		"currentCommandColumn "
+		"shortInstrumentName "
+		"instrumentName "
+		"shortVocalName "
+		"vocalName ",
+
+		/* write */
+		""
+		);