X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fchange-iterator.cc;h=f38344be553d6088cf7def8a8ba24e400f7aea76;hb=3740ac23c9e222602cec4ddc6c38b58504f17673;hp=d36567b5940981cf5ee0dd2babbfcb8b4fce4060;hpb=a2b1b52453dfc35fd3faf16835e18675bd2b2b7f;p=lilypond.git

diff --git a/lily/change-iterator.cc b/lily/change-iterator.cc
index d36567b594..f38344be55 100644
--- a/lily/change-iterator.cc
+++ b/lily/change-iterator.cc
@@ -1,54 +1,65 @@
 /*
-  change-iterator.cc -- implement Change_iterator
+  This file is part of LilyPond, the GNU music typesetter.
 
-  source file of the GNU LilyPond music typesetter
+  Copyright (C) 1997--2015 Han-Wen Nienhuys <hanwen@xs4all.nl>
 
-  (c) 1997--2004 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  LilyPond is free software: you can redistribute it and/or modify
+  it under the terms of the GNU General Public License as published by
+  the Free Software Foundation, either version 3 of the License, or
+  (at your option) any later version.
+
+  LilyPond is distributed in the hope that it will be useful,
+  but WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  GNU General Public License for more details.
+
+  You should have received a copy of the GNU General Public License
+  along with LilyPond.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "change-iterator.hh"
+
 #include "context.hh"
+#include "input.hh"
+#include "international.hh"
 #include "music.hh"
 #include "warn.hh"
-#include "input.hh"
-
 
 void
-Change_iterator::error (String reason)
+Change_iterator::error (const string &reason)
 {
-  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 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;
+  string warn1 = _f ("cannot change `%s' to `%s'", to_type, to_id)
+                 + ": " + reason;
 
   /*
     GUHG!
-   */
-  String warn2= "Change_iterator::process (): " 
-    + get_outlet ()->context_name () + " = `"
-    + get_outlet ()->id_string () + "': ";
+  */
+  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 (Moment m)
 {
-  Context * current = get_outlet ();
-  Context * last = 0;
+  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"));
-
+  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 = *
-   */
+
+  If \translator Staff = bass, then look for Staff = *
+  */
   while (current && !current->is_alias (to_type))
     {
       last = current;
@@ -57,47 +68,44 @@ Change_iterator::process (Moment m)
 
   if (current && current->id_string () == to_id)
     {
-      String msg;
-      msg += _ ("Can't switch translators, I'm there already");
+      string msg;
+      msg += _f ("cannot change, already in translator: %s", to_id);
     }
-  
-  if (current) 
+
+  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
-	  {
-	    get_music ()->origin ()->warning  ("could not find context to switch to.");
-	  }
+        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)
+          {
+            send_stream_event (last, "ChangeParent", get_music ()->origin (),
+                               ly_symbol2scm ("context"), dest->self_scm ());
+          }
+        else
+          /* FIXME: constant error message.  */
+          get_music ()->origin ()->warning (_ ("cannot 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 (_ ("I'm one myself"));
+        /* 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_CTOR_CALLBACK (Change_iterator);