Fix some bugs in the dynamic engraver and PostScript backend

[lilypond.git] / lily / change-iterator.cc

diff --git a/lily/change-iterator.cc b/lily/change-iterator.cc
index c6ebf2907f41548c37853df1d4a1341a08cdbc16..aef7567be0f74e1390b96c55e3b0438b6525b3ba 100644 (file)
--- a/lily/change-iterator.cc
+++ b/lily/change-iterator.cc
@@ -1,43 +1,100 @@
 /*
-  change-iterator.cc -- implement 
+  change-iterator.cc -- implement Change_iterator
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 1997 Han-Wen Nienhuys <hanwen@stack.nl>
+  (c) 1997--2005 Han-Wen Nienhuys <hanwen@xs4all.nl>
 */
 
 #include "change-iterator.hh"
-#include "translator-group.hh"
-#include "change-translator.hh"
 
-Change_iterator::Change_iterator (Change_translator *change_l)
+#include "context.hh"
+#include "input.hh"
+#include "international.hh"
+#include "music.hh"
+#include "warn.hh"
+
+void
+Change_iterator::error (string reason)
 {
-  change_l_ = change_l;
+  string to_type = ly_symbol2string (get_music ()->get_property ("change-to-type"));
+  string to_id = ly_scm2string (get_music ()->get_property ("change-to-id"));
+
+  string warn1 = _f ("can't change `%s' to `%s'", to_type, to_id)
+    + ": " + reason;
+
+  /*
+    GUHG!
+  */
+  string warn2= "Change_iterator::process (): "
+    + get_outlet ()->context_name () + " = `"
+    + get_outlet ()->id_string () + "': ";
+  warning (warn2);
+  get_music ()->origin ()->warning (warn1);
 }
 
 /*
   move to construct_children ?
- */
+*/
 void
-Change_iterator::process_and_next (Moment m)
+Change_iterator::process (Moment m)
 {
-  Translator_group * current = report_to_l ();
-  Translator_group * last = 0;
-  while  (current && current->type_str_ != change_l_->change_to_type_str_)
+  Context *current = get_outlet ();
+  Context *last = 0;
+
+  SCM to_type = get_music ()->get_property ("change-to-type");
+  string to_id = ly_scm2string (get_music ()->get_property ("change-to-id"));
+
+  /* find the type  of translator that we're changing.
+
+  If \translator Staff = bass, then look for Staff = *
+  */
+  while (current && !current->is_alias (to_type))
     {
       last = current;
-      current = current->daddy_trans_l_;
+      current = current->get_parent_context ();
     }
-  if (current) 
+
+  if (current && current->id_string () == to_id)
     {
-      Translator_group * dest = 
-       report_to_l ()->find_create_translator_l (change_l_->change_to_type_str_, 
-                                                 change_l_->change_to_id_str_);
-      current->remove_translator_p (last);
-      dest->add (last);
+      string msg;
+      msg += _f ("can't change, already in translator: %s", to_id);
     }
-  
-  Music_iterator::process_and_next (m);
+
+  if (current)
+    if (last)
+      {
+       Context *dest = 0;
+       Context *where = get_outlet ();
+       while (!dest && where)
+         {
+           dest = find_context_below (where, to_type, to_id);
+           where = where->get_parent_context ();
+         }
+
+       if (dest)
+         {
+           current->remove_context (last);
+           dest->add_context (last);
+         }
+       else
+         /* FIXME: constant error message.  */
+         get_music ()->origin ()->warning (_ ("can't find context to switch to"));
+      }
+    else
+      {
+       /* We could change the current translator's id, but that would make
+          errors hard to catch.
+
+          last->translator_id_string () = get_change
+          ()->change_to_id_string (); */
+       error (_f ("not changing to same context type: %s", ly_symbol2string (to_type).c_str ()));
+      }
+  else
+    /* FIXME: uncomprehensable message */
+    error (_ ("none of these in my family"));
+
+  Simple_music_iterator::process (m);
 }
 
-IMPLEMENT_IS_TYPE_B1 (Change_iterator, Music_iterator);
+IMPLEMENT_CTOR_CALLBACK (Change_iterator);