]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/staff-spacing.cc
* lily/system.cc (do_derived_mark): don't mark from object_alist_
[lilypond.git] / lily / staff-spacing.cc
index 75e6fbedd3bb5f8eae39ed282e68a53556cf2e20..c4ec3b14d605a289962805e1127315c59ad30eb8 100644 (file)
@@ -1,17 +1,16 @@
-/*   
-     staff-spacing.cc -- implement Staff_spacing
-
-     source file of the GNU LilyPond music typesetter
+/*
+  staff-spacing.cc -- implement Staff_spacing
 
-     (c) 2001--2005  Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  source file of the GNU LilyPond music typesetter
 
+  (c) 2001--2005  Han-Wen Nienhuys <hanwen@cs.uu.nl>
 */
 
 #include "staff-spacing.hh"
 
 #include <cstdio>
 
-#include "paper-column.hh" 
+#include "paper-column.hh"
 #include "separation-item.hh"
 #include "warn.hh"
 #include "bar-line.hh"
 #include "note-column.hh"
 #include "stem.hh"
 #include "accidental-placement.hh"
+#include "pointer-group-interface.hh"
 
 /*
   Insert some more space for the next note, in case it has a stem in
   the wrong direction
-
- */
+*/
 Real
-Staff_spacing::next_note_correction (Grob * me,
-                                    Grob * g,
+Staff_spacing::next_note_correction (Grob *me,
+                                    Grob *g,
                                     Interval bar_size)
 {
   if (!g || !Note_column::has_interface (g))
     return 0.0;
 
-  Item *col = dynamic_cast<Item*> (g)->get_column ();
-  Real max_corr = 0. >? (- g->extent (col, X_AXIS)[LEFT]);
+  Item *col = dynamic_cast<Item *> (g)->get_column ();
+  Real max_corr = max (0., (- g->extent (col, X_AXIS)[LEFT]));
 
   /*
     Duh. If this gets out of hand, we should invent something more generic.
-   */
-  if (Grob * a = Note_column::accidentals (g))
+  */
+  if (Grob *a = Note_column::accidentals (g))
     {
       Interval v;
       if (Accidental_placement::has_interface (a))
@@ -48,21 +47,21 @@ Staff_spacing::next_note_correction (Grob * me,
        }
       else
        v = a->extent (col, X_AXIS);
-      
-      max_corr = max_corr >? (- v[LEFT]);
+
+      max_corr = max (max_corr, (- v[LEFT]));
     }
-  if (Grob* a = unsmob_grob (g->get_property ("arpeggio")))
+  if (Grob *a = unsmob_grob (g->get_object ("arpeggio")))
     {
-      max_corr = max_corr >? (- a->extent (col, X_AXIS)[LEFT]);
+      max_corr = max (max_corr, - a->extent (col, X_AXIS)[LEFT]);
     }
-  
+
   /*
     Let's decrease the space a little if the problem is not located
     after a barline.
   */
   if (bar_size.is_empty ())
     max_corr *= 0.75;
-  
+
   if (!bar_size.is_empty ())
     if (Grob *stem = Note_column::get_stem (g))
       {
@@ -70,25 +69,24 @@ Staff_spacing::next_note_correction (Grob * me,
        if (d == DOWN)
          {
            Real stem_start = Stem::head_positions (stem) [DOWN];
-           Real stem_end = Stem::stem_end_position (stem); 
-           Interval stem_posns (stem_start <? stem_end,
-                                stem_end >? stem_start);
+           Real stem_end = Stem::stem_end_position (stem);
+           Interval stem_posns (min (stem_start, stem_end),
+                                max (stem_end, stem_start));
 
            stem_posns.intersect (bar_size);
 
-           Real corr = abs (stem_posns.length ()/7.) <? 1.0;
-           corr *=
-             robust_scm2double (me->get_property ("stem-spacing-correction"), 1);
+           Real corr = min (abs (stem_posns.length () / 7.0), 1.0);
+           corr
+             *= robust_scm2double (me->get_property ("stem-spacing-correction"), 1);
 
            if (d != DOWN)
              corr = 0.0;
-           max_corr = max_corr >? corr;
+           max_corr = max (max_corr, corr);
          }
       }
   return max_corr;
 }
 
-
 /*
   Y-positions that are covered by BAR_GROB, in the case that it is a
   barline.  */
@@ -100,7 +98,7 @@ Staff_spacing::bar_y_positions (Grob *bar_grob)
   if (Bar_line::has_interface (bar_grob))
     {
       SCM glyph = bar_grob->get_property ("glyph");
-      
+
       String glyph_string = scm_is_string (glyph) ? ly_scm2string (glyph) : "";
       if (glyph_string.left_string (1) == "|" || glyph_string.left_string (1) == ".")
        {
@@ -119,44 +117,43 @@ Staff_spacing::bar_y_positions (Grob *bar_grob)
   This is slightly convoluted, since the staffspacing grob gets
   pointers to the separation-items, not the note-columns or
   note-spacings.
-  
- */
+*/
 Real
-Staff_spacing::next_notes_correction (Grob *me, Grob * last_grob)
+Staff_spacing::next_notes_correction (Grob *me, Grob *last_grob)
 {
   Interval bar_size = bar_y_positions (last_grob);
   Real max_corr = 0.0;
 
-  for (SCM s = me->get_property ("right-items");
-       scm_is_pair (s);  s = scm_cdr (s))
+
+  extract_grob_set (me, "right-items", right_items);
+  for (int i = right_items.size (); i--;)
     {
-      Grob * g = unsmob_grob (scm_car (s));
+      Grob *g = right_items[i];
 
-      max_corr = max_corr >?  next_note_correction (me, g,  bar_size);
-      for (SCM t = g->get_property ("elements");
-          scm_is_pair (t); t  = scm_cdr (t))
-       max_corr = max_corr >? next_note_correction (me, unsmob_grob (scm_car (t)), bar_size);
-      
+      max_corr = max (max_corr, next_note_correction (me, g, bar_size));
+      extract_grob_set (g, "elements", elts);
+      for (int j  = elts.size(); j--;)
+       max_corr = max (max_corr, next_note_correction (me, elts[j], bar_size));
     }
-  
+
   return max_corr;
 }
 
 void
-Staff_spacing::get_spacing_params (Grob *me, Real * space, Real * fixed)
+Staff_spacing::get_spacing_params (Grob *me, Real *space, Real *fixed)
 {
   *space = 1.0;
   *fixed = 1.0;
 
-  Grob * separation_item = 0;
-  Item * me_item  = dynamic_cast<Item*> (me);
-    
-  for (SCM s = me->get_property ("left-items");
-       scm_is_pair (s); s = scm_cdr (s))
+  Grob *separation_item = 0;
+  Item *me_item = dynamic_cast<Item *> (me);
+
+  extract_grob_set (me, "left-items", items);
+  for (int i = items.size(); i--;)
     {
-      Grob * cand = unsmob_grob (scm_car (s));
+      Grob *cand = items[i];
       if (cand && Separation_item::has_interface (cand))
-       separation_item = cand ;
+       separation_item = cand;
     }
 
   //  printf ("doing col %d\n" , Paper_column::get_rank (left_col));
@@ -169,30 +166,30 @@ Staff_spacing::get_spacing_params (Grob *me, Real * space, Real * fixed)
 
   Interval last_ext;
   Grob *last_grob = Separation_item::extremal_break_aligned_grob (separation_item, RIGHT,
-                                                &last_ext);
+                                                                 &last_ext);
   if (!last_grob)
     {
       /*
        TODO:
-       
+
        Should  insert a adjustable space here? For excercises, you might want to
-       use a staff without a clef in the beginning. 
-       */
-      
+       use a staff without a clef in the beginning.
+      */
+
       /*
-       we used to have a warning here, but itgenerates a lot of
+       we used to have a warning here, but it generates a lot of
        spurious error messages.
       */
-      return ;
+      return;
     }
 
   *fixed = last_ext[RIGHT];
   *space = *fixed + 1.0;
-  
+
   SCM alist = last_grob->get_property ("space-alist");
   if (!scm_list_p (alist))
-    return ;
-  
+    return;
+
   SCM space_def = scm_sloppy_assq (ly_symbol2scm ("first-note"), alist);
   if (me_item->break_status_dir () == CENTER)
     {
@@ -200,49 +197,47 @@ Staff_spacing::get_spacing_params (Grob *me, Real * space, Real * fixed)
       if (scm_is_pair (nndef))
        space_def = nndef;
     }
-  
-  
+
   if (!scm_is_pair (space_def))
     {
-      programming_error ("Unknown prefatory spacing. "); 
-      return; 
+      programming_error ("unknown prefatory spacing");
+      return;
     }
 
   space_def = scm_cdr (space_def);
   Real distance = scm_to_double (scm_cdr (space_def));
-  SCM type = scm_car (space_def) ;
+  SCM type = scm_car (space_def);
 
   *fixed = last_ext[RIGHT];
   if (type == ly_symbol2scm ("fixed-space"))
     {
-     *fixed += distance;
-     *space = *fixed;
+      *fixed += distance;
+      *space = *fixed;
     }
-  else if (type == ly_symbol2scm ("extra-space")) 
+  else if (type == ly_symbol2scm ("extra-space"))
     {
       *space = *fixed + distance;
     }
   else if (type == ly_symbol2scm ("semi-fixed-space"))
     {
-      *fixed += distance / 2; 
-      *space =  *fixed + distance/2;
+      *fixed += distance / 2;
+      *space = *fixed + distance / 2;
     }
   else if (type == ly_symbol2scm ("minimum-space"))
     {
-      *space = last_ext[LEFT] + (last_ext.length () >? distance);
+      *space = last_ext[LEFT] + max (last_ext.length (), distance);
     }
   else if (type == ly_symbol2scm ("minimum-fixed-space"))
     {
-      *space = last_ext[LEFT] + (last_ext.length () >? distance);
+      *space = last_ext[LEFT] + max (last_ext.length (), distance);
       *fixed = *space;
     }
-  
+
   *space += next_notes_correction (me, last_grob);
 }
 
-
 ADD_INTERFACE (Staff_spacing, "staff-spacing-interface",
               "This object calculates spacing details from a "
               " breakable symbol (left) to another object. For example, it takes care "
               " of  optical spacing from  a bar lines to a note.",
-  "stem-spacing-correction left-items right-items");
+              "stem-spacing-correction left-items right-items");