X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fscript-column.cc;h=b9c9e178f4bc65ddf3e28d3a9ceb76fd0d694bda;hb=d000ba690e920b568ec8272d12de46f92aa20f68;hp=726b32727623e1bd4193b66d713dcf39c47f906b;hpb=d8082113a0df616e7beabc0417c3590a3fac7320;p=lilypond.git

diff --git a/lily/script-column.cc b/lily/script-column.cc
index 726b327276..b9c9e178f4 100644
--- a/lily/script-column.cc
+++ b/lily/script-column.cc
@@ -1,9 +1,20 @@
 /*
-  script-column.cc -- implement Script_column
+  This file is part of LilyPond, the GNU music typesetter.
 
-  source file of the GNU LilyPond music typesetter
+  Copyright (C) 1999--2011 Han-Wen Nienhuys <hanwen@xs4all.nl>
 
-  (c) 1999--2006 Han-Wen Nienhuys <hanwen@xs4all.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 "script-column.hh"
@@ -32,7 +43,7 @@ Script_column::add_side_positioned (Grob *me, Grob *script)
 
 LY_DEFINE (ly_grob_script_priority_less, "ly:grob-script-priority-less",
 	   2, 0, 0, (SCM a, SCM b),
-	   "Compare two grobs by script priority. For internal use.")
+	   "Compare two grobs by script priority.  For internal use.")
 {
   Grob *i1 = unsmob_grob (a);
   Grob *i2 = unsmob_grob (b);
@@ -66,7 +77,7 @@ Script_column::row_before_line_breaking (SCM smob)
 	  affect_all_grobs.push_back (sc);
 	}
       else if (sc->get_property_data ("Y-offset") !=
-	       Side_position_interface::x_aligned_side_proc)
+	       Side_position_interface::y_aligned_side_proc)
 	{
 	  head_scripts_map[sc->get_parent (Y_AXIS)].push_back (sc);
 	}
@@ -105,7 +116,7 @@ Script_column::before_line_breaking (SCM smob)
 	  Side_position_interface::x_aligned_side_proc)
 	staff_sided.push_back (sc);
     }
-  
+
   order_grobs (staff_sided);
   return SCM_UNSPECIFIED;
 }
@@ -128,21 +139,48 @@ Script_column::order_grobs (vector<Grob*> grobs)
       SCM ss = scm_reverse_x (scripts_drul[d], SCM_EOL);
       ss = scm_stable_sort_x (ss, ly_grob_script_priority_less_proc);
 
-      Grob *last = 0;
-      for (SCM s = ss; scm_is_pair (s); s = scm_cdr (s))
-	{
-	  Grob *g = unsmob_grob (scm_car (s));
-	  if (last)
-	    Side_position_interface::add_support (g, last);
+      Grob *g = 0;  // current grob in list
+      Grob *last = 0; // previous grob in list
+      SCM initial_outside_staff = SCM_EOL;  // initial outside_staff_priority of current grob
+      SCM last_initial_outside_staff = SCM_EOL;  // initial outside_staff_priority of previous grob
 
-	  last = g;
+      //  loop over all grobs in script column (already sorted by script_priority)
+      for (SCM s = ss; scm_is_pair (s);
+           s = scm_cdr (s), last = g, last_initial_outside_staff = initial_outside_staff)
+	{
+	  g = unsmob_grob (scm_car (s));
+          initial_outside_staff = g->get_property ("outside-staff-priority");
+          if (last)    //not the first grob in the list
+            {
+	      SCM last_outside_staff = last->get_property ("outside-staff-priority");
+              /*
+                if outside_staff_priority is missing for previous grob, just
+                use it as a support for the current grob
+              */
+	      if (!scm_is_number (last_outside_staff))
+		Side_position_interface::add_support (g, last);
+              /*
+                if outside_staff_priority is missing or is equal to original
+                outside_staff_priority of previous grob, set new
+                outside_staff_priority to just higher than outside_staff_priority
+                of previous grob in order to preserve ordering.
+              */
+              else if ((!scm_is_number (initial_outside_staff)) ||
+                       (fabs (scm_to_double (initial_outside_staff) -
+                             robust_scm2double (last_initial_outside_staff, 0)) < 0.001))
+                g->set_property ("outside-staff-priority",
+				     scm_from_double (
+                                       scm_to_double (last_outside_staff) + 0.1));
+	    }
 	}
     }
   while (flip (&d) != DOWN);
 }
 
 ADD_INTERFACE (Script_column,
-	       "An interface that sorts scripts "
-	       "according to their @code{script-priority}",
-	       
-	       "");
+	       "An interface that sorts scripts according to their"
+	       " @code{script-priority} and @code{outside-staff-priority}.",
+
+	       /* properties */
+	       ""
+	       );