release: 1.3.88

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

diff --git a/lily/change-iterator.cc b/lily/change-iterator.cc
index 81352999e6ebb84cc354e1138a07c02f9c437045..7bd04e1e89f79e2e52b486710f957fea7e276435 100644 (file)
--- a/lily/change-iterator.cc
+++ b/lily/change-iterator.cc
@@ -3,7 +3,7 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c)  1997--1998 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c)  1997--2000 Han-Wen Nienhuys <hanwen@cs.uu.nl>
 */
 
 #include "change-iterator.hh"
@@ -15,35 +15,34 @@
 void
 Change_iterator::error (String reason)
 {
-  String to_type = change_l ()->change_to_type_str_;
-  String to_id =  change_l ()->change_to_id_str_;
+  Change_translator const * t = dynamic_cast<Change_translator const*> (music_l_);   
+  String to_type = t->change_to_type_str_;
+  String to_id =  t->change_to_id_str_;
 
-  String warn1 = _f ("can't change `%s\' to `%s\'", to_type, to_id) 
+  String warn1 = _f ("can't change `%s' to `%s'", to_type, to_id) 
     + ": " + reason;
-  String warn2= "Change_iterator::do_process_and_next (): " 
+  /*
+    GUHG!
+   */
+  String warn2= "Change_iterator::process (): " 
     + report_to_l ()->type_str_ + " = `"
-    + report_to_l ()->id_str_ + "\': ";
+    + report_to_l ()->id_str_ + "': ";
   warning (warn2);
-  change_l ()->warning (warn1);
-}
-
-Change_translator *
-Change_iterator::change_l ()
-{
-  return (Change_translator*) music_l_;
+  t->origin ()->warning (warn1);
 }
 
 /*
   move to construct_children ?
  */
 void
-Change_iterator::do_process_and_next (Moment m)
+Change_iterator::process (Moment m)
 {
   Translator_group * current = report_to_l ();
   Translator_group * last = 0;
 
-  String to_type = change_l ()->change_to_type_str_;
-  String to_id =  change_l ()->change_to_id_str_;
+  Change_translator const * t = dynamic_cast<Change_translator const*> (music_l_); 
+  String to_type = t->change_to_type_str_;
+  String to_id =  t->change_to_id_str_;
 
   /* find the type  of translator that we're changing.
      
@@ -54,7 +53,12 @@ Change_iterator::do_process_and_next (Moment m)
       last = current;
       current = current->daddy_trans_l_;
     }
-  
+
+  if (current && current->id_str_ == to_id)
+    {
+      String msg;
+      msg += _ ("Can't switch translators, I'm there already");
+    }
   
   if (current) 
     if (last)
@@ -62,7 +66,7 @@ Change_iterator::do_process_and_next (Moment m)
        Translator_group * dest = 
          report_to_l ()->find_create_translator_l (to_type, to_id);
        current->remove_translator_p (last);
-       dest->add_translator (last);
+       dest->add_group_translator (last);
       }
     else
       {
@@ -72,11 +76,11 @@ Change_iterator::do_process_and_next (Moment m)
          
           last->translator_id_str_  = change_l ()->change_to_id_str_;
        */
-       error ("I'm one myself");
+       error (_ ("I'm one myself"));
       }
   else
-    error ("None of these in my family.");
-  Music_iterator::do_process_and_next (m);
+    error (_ ("none of these in my family"));
+  Simple_music_iterator::process (m);
 }