]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/note-spacing.cc
Nitpick run.
[lilypond.git] / lily / note-spacing.cc
index 82640d4d626c9db8841537815756c68498fe9263..2bc95d7650420fee548c938ddb913c8e1b7ee79b 100644 (file)
@@ -8,6 +8,7 @@
 
 #include "note-spacing.hh"
 
+#include "grob-array.hh"
 #include "paper-column.hh"
 #include "moment.hh"
 #include "note-column.hh"
@@ -17,6 +18,7 @@
 #include "staff-spacing.hh"
 #include "accidental-placement.hh"
 #include "output-def.hh"
+#include "pointer-group-interface.hh"
 
 /*
   TODO: detect hshifts due to collisions, and account for them in
@@ -27,9 +29,8 @@ void
 Note_spacing::get_spacing (Grob *me, Item *right_col,
                           Real base_space, Real increment, Real *space, Real *fixed)
 {
-
-  Drul_array<SCM> props (me->get_property ("left-items"),
-                        me->get_property ("right-items"));
+  Drul_array<SCM> props (me->get_object ("left-items"),
+                        me->get_object ("right-items"));
   Direction d = LEFT;
   Direction col_dir = right_col->break_status_dir ();
   Drul_array<Interval> extents;
@@ -37,14 +38,14 @@ Note_spacing::get_spacing (Grob *me, Item *right_col,
   Interval left_head_wid;
   do
     {
-      for (SCM s = props[d]; scm_is_pair (s); s = scm_cdr (s))
+      Link_array<Grob> const &items (ly_scm2link_array (props [d]));
+      for (int i = items.size (); i--;)
        {
-         Item *it = dynamic_cast<Item *> (unsmob_grob (scm_car (s)));
+         Item *it = dynamic_cast<Item *> (items[i]);
 
          if (d == RIGHT && it->break_status_dir () != col_dir)
-           {
-             it = it->find_prebroken_piece (col_dir);
-           }
+           it = it->find_prebroken_piece (col_dir);
+
          /*
            some kind of mismatch, eg. a note column, that is behind a
            linebreak.
@@ -64,7 +65,7 @@ Note_spacing::get_spacing (Grob *me, Item *right_col,
 
          if (d == LEFT)
            {
-             SCM r = it->get_property ("rest");
+             SCM r = it->get_object ("rest");
              Grob *g = unsmob_grob (r);
              if (!g)
                g = Note_column::first_head (it);
@@ -91,6 +92,9 @@ Note_spacing::get_spacing (Grob *me, Item *right_col,
 
                  extents[d].unite (v);
                }
+
+             if (Grob *arpeggio = Note_column::arpeggio (it))
+               extents[d].unite (arpeggio->extent (it_col, X_AXIS));
            }
        }
 
@@ -121,9 +125,16 @@ Note_spacing::get_spacing (Grob *me, Item *right_col,
        (extents[LEFT][RIGHT] - left_head_wid[RIGHT]) / 2))
 
     /*
-      What is sticking out of the right note:
+      What is sticking out on the left side of the right note:
     */
-    + (extents[RIGHT].is_empty () ? 0.0 : - extents[RIGHT][LEFT] / 2);
+    + (extents[RIGHT].is_empty ()
+       ? 0.0
+       : ((- extents[RIGHT][LEFT] / 2)
+
+         /*
+           Add that which sticks out a lot.
+         */
+         + max (0.0, -extents[RIGHT][LEFT] - (base_space - increment))));
 
   /*
     We don't do complicated stuff: (base_space - increment) is the
@@ -176,15 +187,15 @@ Note_spacing::right_column (Grob *me)
   if (!me->is_live ())
     return 0;
 
-  SCM right = me->get_property ("right-items");
+  Grob_array *a = unsmob_grob_array (me->get_object ("right-items"));
   Item *mincol = 0;
   int min_rank = INT_MAX;
   bool prune = false;
-  for (SCM s = right; scm_is_pair (s); s = scm_cdr (s))
+  for (int i = 0; a && i < a->size (); i++)
     {
-      Item *ri = unsmob_item (scm_car (s));
-
+      Item *ri = a->item (i);
       Item *col = ri->get_column ();
+
       int rank = Paper_column::get_rank (col);
 
       if (rank < min_rank)
@@ -197,28 +208,18 @@ Note_spacing::right_column (Grob *me)
        }
     }
 
-  if (prune)
+  if (prune && a)
     {
-      // I'm a lazy bum. We could do this in-place.
-      SCM newright = SCM_EOL;
-      for (SCM s = right; scm_is_pair (s); s = scm_cdr (s))
+      Link_array<Grob> &right = a->array_reference ();
+      for (int i = right.size (); i--;)
        {
-         if (unsmob_item (scm_car (s))->get_column () == mincol)
-           newright = scm_cons (scm_car (s), newright);
+         if (dynamic_cast<Item *> (right[i])->get_column () != mincol)
+           right.del (i);
        }
-
-      me->set_property ("right-items", newright);
     }
 
   if (!mincol)
-    {
-      /*
-       int r = Paper_column::get_rank (dynamic_cast<Item*>(me)->get_column ());
-       programming_error (_f ("Spacing wish column %d has no right item.", r));
-      */
-
-      return 0;
-    }
+    return 0;
 
   return mincol;
 }
@@ -238,8 +239,8 @@ Note_spacing::stem_dir_correction (Grob *me, Item *rcolumn,
   Drul_array<Direction> stem_dirs (CENTER, CENTER);
   Drul_array<Interval> stem_posns;
   Drul_array<Interval> head_posns;
-  Drul_array<SCM> props (me->get_property ("left-items"),
-                        me->get_property ("right-items"));
+  Drul_array<SCM> props (me->get_object ("left-items"),
+                        me->get_object ("right-items"));
 
   Drul_array<Spanner *> beams_drul (0, 0);
   Drul_array<Grob *> stems_drul (0, 0);
@@ -255,9 +256,10 @@ Note_spacing::stem_dir_correction (Grob *me, Item *rcolumn,
 
   do
     {
-      for (SCM s = props[d]; scm_is_pair (s); s = scm_cdr (s))
+      Link_array<Grob> const &items (ly_scm2link_array (props [d]));
+      for (int i = 0; i < items.size (); i++)
        {
-         Item *it = dynamic_cast<Item *> (unsmob_grob (scm_car (s)));
+         Item *it = dynamic_cast<Item *> (items[i]);
 
          if (d == RIGHT)
            acc_right = acc_right || Note_column::accidentals (it);
@@ -269,9 +271,7 @@ Note_spacing::stem_dir_correction (Grob *me, Item *rcolumn,
              if (d == RIGHT && Separation_item::has_interface (it))
                {
                  if (it->get_column () != rcolumn)
-                   {
-                     it = it->find_prebroken_piece (rcolumn->break_status_dir ());
-                   }
+                   it = it->find_prebroken_piece (rcolumn->break_status_dir ());
 
                  Grob *last = Separation_item::extremal_break_aligned_grob (it, LEFT, &bar_xextent);
 
@@ -307,9 +307,7 @@ Note_spacing::stem_dir_correction (Grob *me, Item *rcolumn,
          */
          if (d == LEFT
              && Stem::duration_log (stem) > 2 && !Stem::get_beam (stem))
-           {
-             correct_stem_dirs = false;
-           }
+           correct_stem_dirs = false;
 
          Interval hp = Stem::head_positions (stem);
          if (!hp.is_empty ())
@@ -317,7 +315,7 @@ Note_spacing::stem_dir_correction (Grob *me, Item *rcolumn,
              Real chord_start = hp[sd];
              Real stem_end = Stem::stem_end_position (stem);
 
-             stem_posns[d] = Interval (min (chord_start, stem_end), max (chord_start,  stem_end));
+             stem_posns[d] = Interval (min (chord_start, stem_end), max (chord_start, stem_end));
              head_posns[d].unite (hp);
            }
        }
@@ -390,9 +388,7 @@ Note_spacing::stem_dir_correction (Grob *me, Item *rcolumn,
            }
 
          if (!bar_yextent.is_empty ())
-           {
-             correction *= 0.5;
-           }
+           correction *= 0.5;
        }
     }
   else if (correct_stem_dirs && stem_dirs[LEFT] * stem_dirs[RIGHT] == UP)
@@ -426,7 +422,7 @@ Note_spacing::stem_dir_correction (Grob *me, Item *rcolumn,
 
       Real delta = head_posns[-lowest][DOWN] - head_posns[lowest][UP];
       Real corr = robust_scm2double (me->get_property ("same-direction-correction"), 0);
-      
+
       if (delta > 1)
        correction = -lowest * corr;
     }