]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/spacing-spanner.cc
* flower
[lilypond.git] / lily / spacing-spanner.cc
index 389593ff93166d2d8e46271ccd6b42316c7a64dd..7e2f21c72d73e92cc6f8f69c0d2d502a1829e1e7 100644 (file)
@@ -1,11 +1,10 @@
-/*   
+/*
   spacing-spanner.cc -- implement Spacing_spanner
-  
+
   source file of the GNU LilyPond music typesetter
-  
+
   (c) 1999--2005 Han-Wen Nienhuys <hanwen@cs.uu.nl>
-  
- */
+*/
 
 #include <math.h>
 #include <cstdio>
 /*
   TODO: this file/class is too complex. Should figure out how to chop
   this up even more.
-    
- */
+*/
 
 class Spacing_spanner
 {
 public:
-  static void standard_breakable_column_spacing (Grob * me, Item*l, Item*r,
-                                                Real * fixed, Real * space, Moment);
-  
-
-  static Real default_bar_spacing (Grob*, Grob*, Grob*, Moment);
-  static Real note_spacing (Grob*, Grob*, Grob*, Moment, bool*);
-  static Real get_duration_space (Grob*, Moment dur, Rational shortest, bool*);
-  static Rational find_shortest (Grob *, Link_array<Grob> const &);  
-  static void breakable_column_spacing (Grob*, Item* l, Item *r, Moment);
+  static void standard_breakable_column_spacing (Grob *me, Item *l, Item *r,
+                                                Real *fixed, Real *space, Moment);
+
+  static Real default_bar_spacing (Grob *, Grob *, Grob *, Moment);
+  static Real note_spacing (Grob *, Grob *, Grob *, Moment, bool *);
+  static Real get_duration_space (Grob *, Moment dur, Rational shortest, bool *);
+  static Rational find_shortest (Grob *, Link_array<Grob> const &);
+  static void breakable_column_spacing (Grob *, Item *l, Item *r, Moment);
   static void find_loose_columns () {}
-  static void prune_loose_columns (Grob*, Link_array<Grob> *cols, Rational);
+  static void prune_loose_columns (Grob *, Link_array<Grob> *cols, Rational);
   static void find_loose_columns (Link_array<Grob> cols);
   static void set_explicit_neighbor_columns (Link_array<Grob> cols);
   static void set_implicit_neighbor_columns (Link_array<Grob> cols);
-  static void do_measure (Rational, Grob*me, Link_array<Grob> *cols);
-  static void musical_column_spacing (Grob*, Item*, Item*, Real, Rational); 
-  DECLARE_SCHEME_CALLBACK (set_springs, (SCM ));
-  static bool has_interface (Grob*);
+  static void do_measure (Rational, Grob *me, Link_array<Grob> *cols);
+  static void musical_column_spacing (Grob *, Item *, Item *, Real, Rational);
+  DECLARE_SCHEME_CALLBACK (set_springs, (SCM));
+  static bool has_interface (Grob *);
 };
 
 /*
   Return whether COL is fixed to its neighbors by some kind of spacing
   constraint.
 
-  
+
   If in doubt, then we're not loose; the spacing engine should space
   for it, risking suboptimal spacing.
-  
-  (Otherwise, we might risk core dumps, and other weird stuff.)
 
+  (Otherwise, we might risk core dumps, and other weird stuff.)
 */
 static bool
-loose_column (Grob *l, Grob *c, Grob *r) 
+loose_column (Grob *l, Grob *c, Grob *r)
 {
   SCM rns = c->get_property ("right-neighbors");
   SCM lns = c->get_property ("left-neighbors");
 
- /*
 /*
     If this column doesn't have a proper neighbor, we should really
     make it loose, but spacing it correctly is more than we can
     currently can handle.
 
     (this happens in the following situation:
 
-       |
-       |    clef G 
-      *
+    |
+    |    clef G
+    *
 
-       |               |      ||
-       |               |      || 
-      O               O       ||
+    |               |      ||
+    |               |      ||
+    O               O       ||
 
 
     the column containing the clef is really loose, and should be
     attached right to the first column, but that is a lot of work for
     such a borderline case.)
-    
-  */  
+
+  */
   if (!scm_is_pair (lns) || !scm_is_pair (rns))
     return false;
 
-  Item * l_neighbor = dynamic_cast<Item*>  (unsmob_grob (scm_car (lns)));
-  Item * r_neighbor = dynamic_cast<Item*>  (unsmob_grob (scm_car (rns)));
+  Item *l_neighbor = dynamic_cast<Item *> (unsmob_grob (scm_car (lns)));
+  Item *r_neighbor = dynamic_cast<Item *> (unsmob_grob (scm_car (rns)));
 
   if (!l_neighbor || !r_neighbor)
     return false;
 
   l_neighbor = l_neighbor->get_column ();
-  r_neighbor = dynamic_cast<Item*> (Note_spacing::right_column  (r_neighbor));
+  r_neighbor = dynamic_cast<Item *> (Note_spacing::right_column (r_neighbor));
 
   if (l == l_neighbor && r == r_neighbor)
     return false;
@@ -112,28 +108,25 @@ loose_column (Grob *l, Grob *c, Grob *r)
   if (!l_neighbor || !r_neighbor)
     return false;
 
-
-  
   /*
     Only declare loose if the bounds make a little sense.  This means
     some cases (two isolated, consecutive clef changes) won't be
     nicely folded, but hey, then don't do that.
   */
-  if (!  ((Paper_column::is_musical (l_neighbor) || Item::is_breakable (l_neighbor))
-        && (Paper_column::is_musical (r_neighbor) || Item::is_breakable (r_neighbor))) )
+  if (! ((Paper_column::is_musical (l_neighbor) || Item::is_breakable (l_neighbor))
+        && (Paper_column::is_musical (r_neighbor) || Item::is_breakable (r_neighbor))))
     {
       return false;
     }
 
-
   /*
     A rather hairy check, but we really only want to move around clefs. (anything else?)
 
     in any case, we don't want to move bar lines.
-   */
+  */
   for (SCM e = c->get_property ("elements"); scm_is_pair (e); e = scm_cdr (e))
     {
-      Grob * g = unsmob_grob (scm_car (e));
+      Grob *g = unsmob_grob (scm_car (e));
       if (g && Break_align_interface::has_interface (g))
        {
          for (SCM s = g->get_property ("elements"); scm_is_pair (s);
@@ -142,14 +135,14 @@ loose_column (Grob *l, Grob *c, Grob *r)
              Grob *h = unsmob_grob (scm_car (s));
 
              /*
-               ugh. -- fix staff-bar name? 
+               ugh. -- fix staff-bar name?
              */
-             if (h  && h->get_property ("break-align-symbol") == ly_symbol2scm ("staff-bar"))
+             if (h && h->get_property ("break-align-symbol") == ly_symbol2scm ("staff-bar"))
                return false;
            }
        }
     }
-  
+
   return true;
 }
 
@@ -159,11 +152,11 @@ loose_column (Grob *l, Grob *c, Grob *r)
   between.
 */
 void
-Spacing_spanner::prune_loose_columns (Grob*me, Link_array<Grob> *cols, Rational shortest)
+Spacing_spanner::prune_loose_columns (Grob *me, Link_array<Grob> *cols, Rational shortest)
 {
   Link_array<Grob> newcols;
   Real increment = robust_scm2double (me->get_property ("spacing-increment"), 1.2);
-  for (int i = 0; i < cols->size ();  i++)
+  for (int i = 0; i < cols->size (); i++)
     {
       if (Item::is_breakable (cols->elem (i)) || Paper_column::is_musical (cols->elem (i)))
        {
@@ -172,7 +165,7 @@ Spacing_spanner::prune_loose_columns (Grob*me, Link_array<Grob> *cols, Rational
        }
 
       Grob *c = cols->elem (i);
-      if (loose_column (cols->elem (i-1), c, cols->elem (i+1)))
+      if (loose_column (cols->elem (i - 1), c, cols->elem (i + 1)))
        {
          SCM lns = c->get_property ("left-neighbors");
          lns = scm_is_pair (lns) ? scm_car (lns) : SCM_BOOL_F;
@@ -191,17 +184,17 @@ Spacing_spanner::prune_loose_columns (Grob*me, Link_array<Grob> *cols, Rational
          /*
            Set distance constraints for loose columns
          */
-         Drul_array<Grob*> next_door;
+         Drul_array<Grob *> next_door;
          next_door[LEFT] = cols->elem (i - 1);
-         next_door[RIGHT] = cols->elem (i + 1);          
+         next_door[RIGHT] = cols->elem (i + 1);
          Direction d = LEFT;
          Drul_array<Real> dists (0, 0);
 
          do
            {
              dists[d] = 0.0;
-             Item *lc = dynamic_cast<Item*> ((d == LEFT)  ? next_door[LEFT] : c);
-             Item *rc = dynamic_cast<Item*> (d == LEFT  ? c : next_door[RIGHT]);
+             Item *lc = dynamic_cast<Item *> ((d == LEFT) ? next_door[LEFT] : c);
+             Item *rc = dynamic_cast<Item *> (d == LEFT ? c : next_door[RIGHT]);
 
              for (SCM s = lc->get_property ("spacing-wishes");
                   scm_is_pair (s); s = scm_cdr (s))
@@ -220,13 +213,13 @@ Spacing_spanner::prune_loose_columns (Grob*me, Link_array<Grob> *cols, Rational
                      /*
                        The note spacing should be taken from the musical
                        columns.
-                   
+
                      */
                      Real base = note_spacing (me, lc, rc, shortest, &dummy);
                      Note_spacing::get_spacing (sp, rc, base, increment, &space, &fixed);
 
-                     space -= increment; 
-                 
+                     space -= increment;
+
                      dists[d] = dists[d] >? space;
                    }
                  else
@@ -237,15 +230,15 @@ Spacing_spanner::prune_loose_columns (Grob*me, Link_array<Grob> *cols, Rational
 
                      dists[d] = dists[d] >? fixed_space;
                    }
-                 
+
                }
            }
          while (flip (&d) != LEFT);
 
          Rod r;
          r.distance_ = dists[LEFT] + dists[RIGHT];
-         r.item_drul_[LEFT] = dynamic_cast<Item*> (cols->elem (i-1));
-         r.item_drul_[RIGHT] = dynamic_cast<Item*> (cols->elem (i+1));
+         r.item_drul_[LEFT] = dynamic_cast<Item *> (cols->elem (i - 1));
+         r.item_drul_[RIGHT] = dynamic_cast<Item *> (cols->elem (i + 1));
 
          r.add_to_cols ();
        }
@@ -259,7 +252,7 @@ Spacing_spanner::prune_loose_columns (Grob*me, Link_array<Grob> *cols, Rational
 }
 
 /*
-  Set neighboring columns determined by the spacing-wishes grob property.  
+  Set neighboring columns determined by the spacing-wishes grob property.
 */
 void
 Spacing_spanner::set_explicit_neighbor_columns (Link_array<Grob> cols)
@@ -270,21 +263,21 @@ Spacing_spanner::set_explicit_neighbor_columns (Link_array<Grob> cols)
       int min_rank = 100000;   // inf.
 
 
-      SCM wishes =  cols[i]->get_property ("spacing-wishes");
+      SCM wishes = cols[i]->get_property ("spacing-wishes");
       for (SCM s = wishes; scm_is_pair (s); s = scm_cdr (s))
        {
-         Item * wish = dynamic_cast<Item*> (unsmob_grob (scm_car (s)));
+         Item *wish = dynamic_cast<Item *> (unsmob_grob (scm_car (s)));
 
-         Item * lc = wish->get_column ();
-         Grob * right = Note_spacing::right_column (wish);
+         Item *lc = wish->get_column ();
+         Grob *right = Note_spacing::right_column (wish);
 
          if (!right)
            continue;
 
-         Item * rc = dynamic_cast<Item*> (right);
+         Item *rc = dynamic_cast<Item *> (right);
 
          int right_rank = Paper_column::get_rank (rc);
-         int left_rank = Paper_column::get_rank (lc);    
+         int left_rank = Paper_column::get_rank (lc);
 
          /*
            update the left column.
@@ -292,8 +285,8 @@ Spacing_spanner::set_explicit_neighbor_columns (Link_array<Grob> cols)
          if (right_rank <= min_rank)
            {
              if (right_rank < min_rank)
-               right_neighbors  = SCM_EOL;
-             
+               right_neighbors = SCM_EOL;
+
              min_rank = right_rank;
              right_neighbors = scm_cons (wish->self_scm (), right_neighbors);
            }
@@ -306,7 +299,7 @@ Spacing_spanner::set_explicit_neighbor_columns (Link_array<Grob> cols)
          if (scm_is_pair (left_neighs)
              && unsmob_grob (scm_car (left_neighs)))
            {
-             Item * it = dynamic_cast<Item*> (unsmob_grob (scm_car (left_neighs)));
+             Item *it = dynamic_cast<Item *> (unsmob_grob (scm_car (left_neighs)));
              maxrank = Paper_column::get_rank (it->get_column ());
            }
 
@@ -329,14 +322,14 @@ Spacing_spanner::set_explicit_neighbor_columns (Link_array<Grob> cols)
 
 /*
   Set neighboring columns that have no left/right-neighbor set
-  yet. Only do breakable non-musical columns, and musical columns. 
+  yet. Only do breakable non-musical columns, and musical columns.
 */
 void
 Spacing_spanner::set_implicit_neighbor_columns (Link_array<Grob> cols)
 {
   for (int i = 0; i < cols.size (); i++)
     {
-      Item * it = dynamic_cast<Item*>(cols[i]);
+      Item *it = dynamic_cast<Item *> (cols[i]);
       if (!Item::is_breakable (it) && !Paper_column::is_musical (it))
        continue;
 
@@ -346,20 +339,19 @@ Spacing_spanner::set_implicit_neighbor_columns (Link_array<Grob> cols)
        sloppy with typnig left/right-neighbors should take list, but paper-column found instead.
       */
       SCM ln = cols[i] ->get_property ("left-neighbors");
-      if (!scm_is_pair (ln) && i ) 
+      if (!scm_is_pair (ln) && i)
        {
-         cols[i]->set_property ("left-neighbors", scm_cons (cols[i-1]->self_scm (), SCM_EOL));
+         cols[i]->set_property ("left-neighbors", scm_cons (cols[i - 1]->self_scm (), SCM_EOL));
        }
 
       SCM rn = cols[i] ->get_property ("right-neighbors");
-      if (!scm_is_pair (rn) && i < cols.size () - 1) 
+      if (!scm_is_pair (rn) && i < cols.size () - 1)
        {
          cols[i]->set_property ("right-neighbors", scm_cons (cols[i + 1]->self_scm (), SCM_EOL));
        }
     }
 }
 
-
 MAKE_SCHEME_CALLBACK (Spacing_spanner, set_springs, 1);
 SCM
 Spacing_spanner::set_springs (SCM smob)
@@ -388,23 +380,21 @@ Spacing_spanner::set_springs (SCM smob)
   prune_loose_columns (me, &all, global_shortest);
   set_implicit_neighbor_columns (all);
 
-  
   int j = 0;
   for (int i = 1; i < all.size (); i++)
     {
       Grob *sc = all[i];
       if (Item::is_breakable (sc))
-        {
-         Link_array<Grob> measure (all.slice (j, i+1));          
-          do_measure (global_shortest, me, &measure);
+       {
+         Link_array<Grob> measure (all.slice (j, i + 1));
+         do_measure (global_shortest, me, &measure);
          j = i;
-        }
+       }
     }
 
   return SCM_UNSPECIFIED;
 }
 
-
 /*
   We want the shortest note that is also "common" in the piece, so we
   find the shortest in each measure, and take the most frequently
@@ -413,7 +403,6 @@ Spacing_spanner::set_springs (SCM smob)
   This probably gives weird effects with modern music, where every
   note has a different duration, but hey, don't write that kind of
   stuff, then.
-
 */
 Rational
 Spacing_spanner::find_shortest (Grob *me, Link_array<Grob> const &cols)
@@ -421,25 +410,25 @@ Spacing_spanner::find_shortest (Grob *me, Link_array<Grob> const &cols)
   /*
     ascending in duration
   */
-  Array<Rational> durations; 
+  Array<Rational> durations;
   Array<int> counts;
-  
+
   Rational shortest_in_measure;
   shortest_in_measure.set_infinite (1);
-  
-  for (int i = 0 ; i < cols.size (); i++)  
+
+  for (int i = 0; i < cols.size (); i++)
     {
       if (Paper_column::is_musical (cols[i]))
        {
-         Moment *when = unsmob_moment (cols[i]->get_property  ("when"));
+         Moment *when = unsmob_moment (cols[i]->get_property ("when"));
 
          /*
            ignore grace notes for shortest notes.
          */
          if (when && when->grace_part_)
            continue;
-         
-         SCM  st = cols[i]->get_property ("shortest-starter-duration");
+
+         SCM st = cols[i]->get_property ("shortest-starter-duration");
          Moment this_shortest = *unsmob_moment (st);
          assert (this_shortest.to_bool ());
          shortest_in_measure = shortest_in_measure <? this_shortest.main_part_;
@@ -469,7 +458,7 @@ Spacing_spanner::find_shortest (Grob *me, Link_array<Grob> const &cols)
              counts.push (1);
            }
 
-         shortest_in_measure.set_infinite (1); 
+         shortest_in_measure.set_infinite (1);
        }
     }
 
@@ -486,13 +475,13 @@ Spacing_spanner::find_shortest (Grob *me, Link_array<Grob> const &cols)
       //      printf ("duration %d/%d, count %d\n", durations[i].num (), durations[i].den (), counts[i]);
     }
 
-  SCM  bsd = me->get_property ("base-shortest-duration");
+  SCM bsd = me->get_property ("base-shortest-duration");
   Rational d = Rational (1, 8);
   if (Moment *m = unsmob_moment (bsd))
     d = m->main_part_;
-  
+
   if (max_idx >= 0)
-    d = d <? durations[max_idx] ;
+    d = d <? durations[max_idx];
 
   return d;
 }
@@ -502,35 +491,34 @@ Spacing_spanner::find_shortest (Grob *me, Link_array<Grob> const &cols)
   per-measure spacing. Now we have piecewise spacing. We should fix
   this to support "spacing-regions": some regions have different notes
   (different time sigs) than others, and should be spaced differently.
- */
+*/
 void
-Spacing_spanner::do_measure (Rational global_shortest, Grob*me,
-                            Link_array<Grob> *cols) 
+Spacing_spanner::do_measure (Rational global_shortest, Grob *me,
+                            Link_array<Grob> *cols)
 {
 
   Real headwid = robust_scm2double (me->get_property ("spacing-increment"), 1);
   for (int i = 0; i < cols->size () - 1; i++)
     {
-      Item * l = dynamic_cast<Item*> (cols->elem (i));
-      Item * r =  dynamic_cast<Item*> (cols->elem (i+1));
+      Item *l = dynamic_cast<Item *> (cols->elem (i));
+      Item *r = dynamic_cast<Item *> (cols->elem (i + 1));
 
-      Paper_column * lc = dynamic_cast<Paper_column*> (l);
-      Paper_column * rc = dynamic_cast<Paper_column*> (r);
+      Paper_column *lc = dynamic_cast<Paper_column *> (l);
+      Paper_column *rc = dynamic_cast<Paper_column *> (r);
 
       if (!Paper_column::is_musical (l))
        {
          breakable_column_spacing (me, l, r, global_shortest);
 
          /*
-           
            The case that the right part is broken as well is rather
            rare, but it is possible, eg. with a single empty measure,
            or if one staff finishes a tad earlier than the rest.
-           
-          */
+
+         */
          Item *lb = l->find_prebroken_piece (RIGHT);
          Item *rb = r->find_prebroken_piece (LEFT);
-         
+
          if (lb)
            breakable_column_spacing (me, lb, r, global_shortest);
 
@@ -544,16 +532,16 @@ Spacing_spanner::do_measure (Rational global_shortest, Grob*me,
          musical_column_spacing (me, lc, rc, headwid, global_shortest);
          if (Item *rb = r->find_prebroken_piece (LEFT))
            musical_column_spacing (me, lc, rb, headwid, global_shortest);
-       }    
+       }
     }
 }
 
 /*
   Generate the space between two musical columns LC and RC, given
   spacing parameters INCR and SHORTEST.
- */
+*/
 void
-Spacing_spanner::musical_column_spacing (Grob *me, Item * lc, Item *rc, Real increment, Rational global_shortest)
+Spacing_spanner::musical_column_spacing (Grob *me, Item *lc, Item *rc, Real increment, Rational global_shortest)
 {
   bool expand_only = false;
   Real base_note_space = note_spacing (me, lc, rc, global_shortest, &expand_only);
@@ -561,8 +549,8 @@ Spacing_spanner::musical_column_spacing (Grob *me, Item * lc, Item *rc, Real inc
   Real compound_note_space = 0.0;
   Real compound_fixed_note_space = 0.0;
   int wish_count = 0;
-  
-  SCM seq  = lc->get_property ("right-neighbors");
+
+  SCM seq = lc->get_property ("right-neighbors");
 
   /*
     We adjust the space following a note only if the next note
@@ -571,7 +559,7 @@ Spacing_spanner::musical_column_spacing (Grob *me, Item * lc, Item *rc, Real inc
   */
   for (SCM s = seq; scm_is_pair (s); s = scm_cdr (s))
     {
-      Grob * wish = unsmob_grob (scm_car (s));
+      Grob *wish = unsmob_grob (scm_car (s));
 
       Item *wish_rcol = Note_spacing::right_column (wish);
       if (Note_spacing::left_column (wish) != lc
@@ -585,30 +573,29 @@ Spacing_spanner::musical_column_spacing (Grob *me, Item * lc, Item *rc, Real inc
        {
          Real space = 0.0;
          Real fixed = 0.0;
-         
+
          Note_spacing::get_spacing (wish, rc, base_note_space, increment, &space, &fixed);
 
-         
          compound_note_space = compound_note_space + space;
          compound_fixed_note_space = compound_fixed_note_space + fixed;
-         wish_count ++;
-         
+         wish_count++;
+
        }
     }
 
-  if (Paper_column::when_mom (rc).grace_part_ &&
-      !Paper_column::when_mom (lc).grace_part_)
+  if (Paper_column::when_mom (rc).grace_part_
+      && !Paper_column::when_mom (lc).grace_part_)
     {
       /*
        Ugh. 0.8 is arbitrary.
-       */
-      compound_note_space *= 0.8; 
+      */
+      compound_note_space *= 0.8;
     }
-  
+
   if (compound_note_space < 0 || wish_count == 0)
     {
       compound_note_space = base_note_space;
-      compound_fixed_note_space =  increment;
+      compound_fixed_note_space = increment;
     }
   else
     {
@@ -630,7 +617,7 @@ Spacing_spanner::musical_column_spacing (Grob *me, Item * lc, Item *rc, Real inc
 
   /*
     TODO: make sure that the space doesn't exceed the right margin.
-   */
+  */
   if (packed)
     {
       /*
@@ -651,23 +638,22 @@ Spacing_spanner::musical_column_spacing (Grob *me, Item * lc, Item *rc, Real inc
       distance = compound_note_space;
     }
 
-  Spaceable_grob::add_spring (lc, rc, distance, strength);  
+  Spaceable_grob::add_spring (lc, rc, distance, strength);
 }
 
-
 /*
   The one-size-fits all spacing. It doesn't take into account
   different spacing wishes from one to the next column.
- */
+*/
 void
-Spacing_spanner::standard_breakable_column_spacing (Grob * me, Item*l, Item*r,
-                                  Real * fixed, Real * space,
-                                  Moment shortest)
+Spacing_spanner::standard_breakable_column_spacing (Grob *me, Item *l, Item *r,
+                                                   Real *fixed, Real *space,
+                                                   Moment shortest)
 {
   *fixed = 0.0;
   Direction d = LEFT;
-  Drul_array<Item*> cols (l, r);
-  
+  Drul_array<Item *> cols (l, r);
+
   do
     {
       if (!Paper_column::is_musical (cols[d]))
@@ -675,14 +661,13 @@ Spacing_spanner::standard_breakable_column_spacing (Grob * me, Item*l, Item*r,
          /*
            Tied accidentals over barlines cause problems, so lets see
            what happens if we do this for non musical columns only.
-          */
+         */
          Interval lext = cols[d]->extent (cols [d], X_AXIS);
          if (!lext.is_empty ())
            *fixed += -d * lext[-d];
        }
     }
   while (flip (&d) != LEFT);
-  
 
   if (l->is_breakable (l) && r->is_breakable (r))
     {
@@ -690,10 +675,10 @@ Spacing_spanner::standard_breakable_column_spacing (Grob * me, Item*l, Item*r,
       Moment mlen (1);
       if (dt)
        mlen = *dt;
-      
+
       Real incr = robust_scm2double (me->get_property ("spacing-increment"), 1);
 
-      *space =  *fixed + incr * double (mlen.main_part_ / shortest.main_part_) * 0.8;
+      *space = *fixed + incr * double (mlen.main_part_ / shortest.main_part_) * 0.8;
     }
   else
     {
@@ -704,8 +689,8 @@ Spacing_spanner::standard_breakable_column_spacing (Grob * me, Item*l, Item*r,
          /*
            In this case, Staff_spacing should handle the job,
            using dt when it is 0 is silly.
-          */
-         *space = *fixed + 0.5; 
+         */
+         *space = *fixed + 0.5;
        }
       else
        {
@@ -715,12 +700,11 @@ Spacing_spanner::standard_breakable_column_spacing (Grob * me, Item*l, Item*r,
     }
 }
 
-
 /*
   Read hints from L and generate springs.
 */
 void
-Spacing_spanner::breakable_column_spacing (Grob*me, Item* l, Item *r, Moment shortest)
+Spacing_spanner::breakable_column_spacing (Grob *me, Item *l, Item *r, Moment shortest)
 {
   Real compound_fixed = 0.0;
   Real compound_space = 0.0;
@@ -733,7 +717,7 @@ Spacing_spanner::breakable_column_spacing (Grob*me, Item* l, Item *r, Moment sho
       for (SCM s = l->get_property ("spacing-wishes");
           scm_is_pair (s); s = scm_cdr (s))
        {
-         Item * spacing_grob = dynamic_cast<Item*> (unsmob_grob (scm_car (s)));
+         Item *spacing_grob = dynamic_cast<Item *> (unsmob_grob (scm_car (s)));
 
          if (!spacing_grob || !Staff_spacing::has_interface (spacing_grob))
            continue;
@@ -763,13 +747,13 @@ Spacing_spanner::breakable_column_spacing (Grob*me, Item* l, Item *r, Moment sho
 
          compound_space += space;
          compound_fixed += fixed_space;
-         wish_count ++ ;
+         wish_count++;
        }
     }
 
   if (compound_space <= 0.0 || !wish_count)
     {
-      standard_breakable_column_spacing (me, l, r, &compound_fixed, &compound_space ,
+      standard_breakable_column_spacing (me, l, r, &compound_fixed, &compound_space,
                                         shortest);
       wish_count = 1;
     }
@@ -782,34 +766,32 @@ Spacing_spanner::breakable_column_spacing (Grob*me, Item* l, Item *r, Moment sho
   assert (!isinf (compound_space));
   compound_space = compound_space >? compound_fixed;
 
-  
   /*
     Hmm.  we do 1/0 in the next thing. Perhaps we should check if this
     works on all architectures.
-   */
+  */
 
   /*
     There used to be code that changed spacing depending on
     raggedright setting.  Ugh.
 
-    Do it more cleanly, or rename the property. 
-    
-   */
+    Do it more cleanly, or rename the property.
+
+  */
   Real strength = 1 / (compound_space - compound_fixed);
   Real distance = compound_space;
   Spaceable_grob::add_spring (l, r, distance, strength);
 }
 
-
 /**
-  Get the measure wide ant for arithmetic spacing.
-  */
+   Get the measure wide ant for arithmetic spacing.
+*/
 Real
-Spacing_spanner::get_duration_space (Grob*me, Moment d, Rational shortest, bool * expand_only) 
+Spacing_spanner::get_duration_space (Grob *me, Moment d, Rational shortest, bool *expand_only)
 {
   Real k = robust_scm2double (me->get_property ("shortest-duration-space"), 1);
   Real incr = robust_scm2double (me->get_property ("spacing-increment"), 1);
-  
+
   if (d < shortest)
     {
       /*
@@ -825,39 +807,39 @@ Spacing_spanner::get_duration_space (Grob*me, Moment d, Rational shortest, bool
        mm for 8th. (white space), suggesting that we use
 
        (1.2 / 1.5)^{-log2(duration ratio)}
-       
 
-       */
+
+      */
       Rational ratio = d.main_part_ / shortest;
 
-      return ((k-1) + double (ratio)) * incr;
+      return ((k - 1) + double (ratio)) * incr;
     }
   else
     {
       /*
-         John S. Gourlay. ``Spacing a Line of Music, '' Technical
-         Report OSU-CISRC-10/87-TR35, Department of Computer and
-         Information Science, The Ohio State University, 1987.
-       */
-      Real log =  log_2 (shortest);
+       John S. Gourlay. ``Spacing a Line of Music, '' Technical
+       Report OSU-CISRC-10/87-TR35, Department of Computer and
+       Information Science, The Ohio State University, 1987.
+      */
+      Real log = log_2 (shortest);
       k -= log;
       Rational compdur = d.main_part_ + d.grace_part_ /Rational (3);
-      *expand_only = false;      
-   
+      *expand_only = false;
+
       return (log_2 (compdur) + k) * incr;
     }
 }
 
 Real
-Spacing_spanner::note_spacing (Grob*me, Grob *lc, Grob *rc,
-                              Moment shortest, bool * expand_only) 
+Spacing_spanner::note_spacing (Grob *me, Grob *lc, Grob *rc,
+                              Moment shortest, bool *expand_only)
 {
   Moment shortest_playing_len = 0;
   SCM s = lc->get_property ("shortest-playing-duration");
 
   if (unsmob_moment (s))
     shortest_playing_len = *unsmob_moment (s);
-  
+
   if (! shortest_playing_len.to_bool ())
     {
       programming_error ("can't find a ruling note at " + Paper_column::when_mom (lc).to_string ());
@@ -865,7 +847,7 @@ Spacing_spanner::note_spacing (Grob*me, Grob *lc, Grob *rc,
     }
 
   Moment lwhen = Paper_column::when_mom (lc);
-  Moment rwhen =  Paper_column::when_mom (rc);
+  Moment rwhen = Paper_column::when_mom (rc);
 
   Moment delta_t = rwhen - lwhen;
   if (!Paper_column::is_musical (rc))
@@ -874,8 +856,8 @@ Spacing_spanner::note_spacing (Grob*me, Grob *lc, Grob *rc,
        when toying with mmrests, it is possible to have musical
        column on the left and non-musical on the right, spanning
        several measures.
-       */
-      
+      */
+
       Moment *dt = unsmob_moment (rc->get_property ("measure-length"));
       if (dt)
        {
@@ -884,7 +866,7 @@ Spacing_spanner::note_spacing (Grob*me, Grob *lc, Grob *rc,
          /*
            The following is an extra safety measure, such that
            the length of a mmrest event doesn't cause havoc.
-          */
+         */
          shortest_playing_len = shortest_playing_len <? *dt;
        }
     }
@@ -895,8 +877,8 @@ Spacing_spanner::note_spacing (Grob*me, Grob *lc, Grob *rc,
     SHORTEST_PLAYING_LEN away. However chord-tremolos do funky faking stuff
     with durations, invalidating this assumption. Here we kludge
     around to get chord tremolos to behave properly.
-    
-   */
+
+  */
   shortest_playing_len = shortest_playing_len >? delta_t;
   if (delta_t.main_part_ && !lwhen.grace_part_)
     {
@@ -916,28 +898,23 @@ Spacing_spanner::note_spacing (Grob*me, Grob *lc, Grob *rc,
       dist *= grace_fact;
     }
 
-  
   return dist;
 }
 
-
-
 ADD_INTERFACE (Spacing_spanner, "spacing-spanner-interface",
-"The space taken by a note is dependent on its duration. Doubling a\n"
-"duration adds spacing-increment to the space. The most common shortest\n"
-"note gets @code{shortest-duration-space}. Notes that are even shorter are\n"
-"spaced proportonial to their duration.\n"
-"\n"
-"Typically, the increment is the width of a black note head.  In a\n"
-"piece with lots of 8th notes, and some 16th notes, the eighth note\n"
-"gets 2 note heads width (i.e. the space following a note is 1 note\n"
-"head width) A 16th note is followed by 0.5 note head width. The\n"
-"quarter note is followed by  3 NHW, the half by 4 NHW, etc.\n",
-  "grace-space-factor spacing-increment base-shortest-duration shortest-duration-space common-shortest-duration");
-
-
+              "The space taken by a note is dependent on its duration. Doubling a\n"
+              "duration adds spacing-increment to the space. The most common shortest\n"
+              "note gets @code{shortest-duration-space}. Notes that are even shorter are\n"
+              "spaced proportonial to their duration.\n"
+              "\n"
+              "Typically, the increment is the width of a black note head.  In a\n"
+              "piece with lots of 8th notes, and some 16th notes, the eighth note\n"
+              "gets 2 note heads width (i.e. the space following a note is 1 note\n"
+              "head width) A 16th note is followed by 0.5 note head width. The\n"
+              "quarter note is followed by  3 NHW, the half by 4 NHW, etc.\n",
+              "grace-space-factor spacing-increment base-shortest-duration shortest-duration-space common-shortest-duration");
 
 ADD_INTERFACE (Spacing_interface, "spacing-interface",
-  "Something to do with line breaking and spacing. Kill this one after determining line breaks.",
-  "");
+              "Something to do with line breaking and spacing. Kill this one after determining line breaks.",
+              "");