]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/axis-group-interface.cc
Issue 2491: Macro for(UP_and_DOWN) and 3 similar.
[lilypond.git] / lily / axis-group-interface.cc
index 07ed23f64ff025f26e4e5464d9078694b8f3fad8..1b937985eeee2aab63444c47401f2a49e9a59af7 100644 (file)
@@ -1,27 +1,44 @@
 /*
-  axis-group-interface.cc -- implement Axis_group_interface
+  This file is part of LilyPond, the GNU music typesetter.
 
-  source file of the GNU LilyPond music typesetter
+  Copyright (C) 2000--2012 Han-Wen Nienhuys <hanwen@xs4all.nl>
 
-  (c) 2000--2009 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 "axis-group-interface.hh"
 
 #include "align-interface.hh"
 #include "directional-element-interface.hh"
-#include "pointer-group-interface.hh"
 #include "grob-array.hh"
 #include "hara-kiri-group-spanner.hh"
 #include "international.hh"
 #include "lookup.hh"
 #include "paper-column.hh"
 #include "paper-score.hh"
+#include "pointer-group-interface.hh"
 #include "separation-item.hh"
 #include "skyline-pair.hh"
+#include "staff-grouper-interface.hh"
+#include "stem.hh"
 #include "stencil.hh"
 #include "system.hh"
 #include "warn.hh"
+#include "unpure-pure-container.hh"
+
+static bool
+pure_staff_priority_less (Grob *const &g1, Grob *const &g2);
 
 void
 Axis_group_interface::add_element (Grob *me, Grob *e)
@@ -35,12 +52,12 @@ Axis_group_interface::add_element (Grob *me, Grob *e)
       Axis a = (Axis) scm_to_int (scm_car (ax));
 
       if (!e->get_parent (a))
-       e->set_parent (me, a);
+        e->set_parent (me, a);
 
       e->set_object ((a == X_AXIS)
-                    ? ly_symbol2scm ("axis-group-parent-X")
-                    : ly_symbol2scm ("axis-group-parent-Y"),
-                    me->self_scm ());
+                     ? ly_symbol2scm ("axis-group-parent-X")
+                     : ly_symbol2scm ("axis-group-parent-Y"),
+                     me->self_scm ());
     }
 
   /* must be ordered, because Align_interface also uses
@@ -57,45 +74,109 @@ Axis_group_interface::has_axis (Grob *me, Axis a)
 }
 
 Interval
-Axis_group_interface::relative_group_extent (vector<Grob*> const &elts,
-                                            Grob *common, Axis a)
+Axis_group_interface::relative_group_extent (vector<Grob *> const &elts,
+                                             Grob *common, Axis a)
+{
+  return relative_maybe_bound_group_extent (elts, common, a, false);
+}
+
+Interval
+Axis_group_interface::relative_maybe_bound_group_extent (vector<Grob *> const &elts,
+                                                         Grob *common, Axis a, bool bound)
 {
   Interval r;
   for (vsize i = 0; i < elts.size (); i++)
     {
       Grob *se = elts[i];
       if (!to_boolean (se->get_property ("cross-staff")))
-       {
-         Interval dims = se->extent (common, a);
-         if (!dims.is_empty ())
-           r.unite (dims);
-       }
+        {
+          Interval dims = (bound && has_interface (se)
+                           ? generic_bound_extent (se, common, a)
+                           : se->extent (common, a));
+          if (!dims.is_empty ())
+            r.unite (dims);
+        }
     }
   return r;
 }
 
 Interval
-Axis_group_interface::cached_pure_height (Grob *me, int start, int end)
+Axis_group_interface::generic_bound_extent (Grob *me, Grob *common, Axis a)
 {
-  SCM adjacent_pure_heights = me->get_property ("adjacent-pure-heights");
+  /* trigger the callback to do skyline-spacing on the children */
+  if (a == Y_AXIS)
+    (void) me->get_property ("vertical-skylines");
+
+  extract_grob_set (me, "elements", elts);
+  vector<Grob *> new_elts;
+
+  SCM interfaces = me->get_property ("bound-alignment-interfaces");
+
+  for (vsize i = 0; i < elts.size (); i++)
+    for (SCM l = interfaces; scm_is_pair (l); l = scm_cdr (l))
+      if (elts[i]->internal_has_interface (scm_car (l)))
+        new_elts.push_back (elts[i]);
 
-  if (!scm_is_pair (adjacent_pure_heights)
-      || !scm_is_vector (scm_cdr (adjacent_pure_heights)))
-    return Interval (0, 0);
+  if (!new_elts.size ())
+    return robust_relative_extent (me, common, a);
 
-  return combine_pure_heights (me, scm_cdr (adjacent_pure_heights), start, end);
+  if (!common)
+    common = common_refpoint_of_array (new_elts, me, a);
+
+  return relative_maybe_bound_group_extent (new_elts, common, a, true);
 }
 
 Interval
-Axis_group_interface::begin_of_line_pure_height (Grob *me, int start)
+Axis_group_interface::sum_partial_pure_heights (Grob *me, int start, int end)
 {
+  Interval iv = begin_of_line_pure_height (me, start);
+  iv.unite (rest_of_line_pure_height (me, start, end));
+
+  return iv;
+}
+
+Interval
+Axis_group_interface::part_of_line_pure_height (Grob *me, bool begin, int start, int end)
+{
+  Spanner *sp = dynamic_cast<Spanner *> (me);
+  SCM cache_symbol = begin
+                     ? ly_symbol2scm ("begin-of-line-pure-height")
+                     : ly_symbol2scm ("rest-of-line-pure-height");
+  SCM cached = sp->get_cached_pure_property (cache_symbol, start, end);
+  if (scm_is_pair (cached))
+    return robust_scm2interval (cached, Interval (0, 0));
+
   SCM adjacent_pure_heights = me->get_property ("adjacent-pure-heights");
+  Interval ret;
+
+  if (!scm_is_pair (adjacent_pure_heights))
+    ret = Interval (0, 0);
+  else
+    {
+      SCM these_pure_heights = begin
+                               ? scm_car (adjacent_pure_heights)
+                               : scm_cdr (adjacent_pure_heights);
+
+      if (scm_is_vector (these_pure_heights))
+        ret = combine_pure_heights (me, these_pure_heights, start, end);
+      else
+        ret = Interval (0, 0);
+    }
+
+  sp->cache_pure_property (cache_symbol, start, end, ly_interval2scm (ret));
+  return ret;
+}
 
-  if (!scm_is_pair (adjacent_pure_heights)
-      || !scm_is_vector (scm_car (adjacent_pure_heights)))
-    return Interval (0, 0);
+Interval
+Axis_group_interface::begin_of_line_pure_height (Grob *me, int start)
+{
+  return part_of_line_pure_height (me, true, start, start + 1);
+}
 
-  return combine_pure_heights (me, scm_car (adjacent_pure_heights), start, start+1);
+Interval
+Axis_group_interface::rest_of_line_pure_height (Grob *me, int start, int end)
+{
+  return part_of_line_pure_height (me, false, start, end);
 }
 
 Interval
@@ -103,17 +184,17 @@ Axis_group_interface::combine_pure_heights (Grob *me, SCM measure_extents, int s
 {
   Paper_score *ps = get_root_system (me)->paper_score ();
   vector<vsize> breaks = ps->get_break_indices ();
-  vector<Grob*> cols = ps->get_columns ();
+  vector<Grob *> cols = ps->get_columns ();
 
   Interval ext;
   for (vsize i = 0; i + 1 < breaks.size (); i++)
     {
       int r = Paper_column::get_rank (cols[breaks[i]]);
       if (r >= end)
-       break;
+        break;
 
       if (r >= start)
-       ext.unite (ly_scm2interval (scm_c_vector_ref (measure_extents, i)));
+        ext.unite (ly_scm2interval (scm_c_vector_ref (measure_extents, i)));
     }
 
   return ext;
@@ -125,70 +206,100 @@ Axis_group_interface::combine_pure_heights (Grob *me, SCM measure_extents, int s
 // is at the beginning of a line. The second vector stores, for each measure,
 // the combined height of the elements that are present only when the bar
 // is not at the beginning of a line.
-
 MAKE_SCHEME_CALLBACK (Axis_group_interface, adjacent_pure_heights, 1)
 SCM
 Axis_group_interface::adjacent_pure_heights (SCM smob)
 {
   Grob *me = unsmob_grob (smob);
 
-  Grob *common = calc_pure_elts_and_common (me);
-  extract_grob_set (me, "pure-relevant-items", items);
-  extract_grob_set (me, "pure-relevant-spanners", spanners);
+  Grob *common = unsmob_grob (me->get_object ("pure-Y-common"));
+  extract_grob_set (me, "pure-relevant-grobs", elts);
 
   Paper_score *ps = get_root_system (me)->paper_score ();
-  vector<vsize> breaks = ps->get_break_indices ();
-  vector<Grob*> cols = ps->get_columns ();
+  vector<vsize> ranks = ps->get_break_ranks ();
 
-  SCM begin_line_heights = scm_c_make_vector (breaks.size () - 1, SCM_EOL);
-  SCM mid_line_heights = scm_c_make_vector (breaks.size () - 1, SCM_EOL);
+  vector<Interval> begin_line_heights;
+  vector<Interval> mid_line_heights;
+  vector<Interval> begin_line_staff_heights;
+  vector<Interval> mid_line_staff_heights;
+  begin_line_heights.resize (ranks.size () - 1);
+  mid_line_heights.resize (ranks.size () - 1);
 
-  vsize it_index = 0;
-  for (vsize i = 0; i + 1 < breaks.size (); i++)
+  for (vsize i = 0; i < elts.size (); ++i)
+    {
+      Grob *g = elts[i];
+
+      if (to_boolean (g->get_property ("cross-staff")))
+        continue;
+
+      bool outside_staff = scm_is_number (g->get_property ("outside-staff-priority"));
+      Real padding = robust_scm2double (g->get_property ("outside-staff-padding"), 0.5);
+
+      // When we encounter the first outside-staff grob, make a copy
+      // of the current heights to use as an estimate for the staff heights.
+      // Note that the outside-staff approximation that we use here doesn't
+      // consider any collisions that might occur between outside-staff grobs,
+      // but only the fact that outside-staff grobs may need to be raised above
+      // the staff.
+      if (outside_staff && begin_line_staff_heights.empty ())
+        {
+          begin_line_staff_heights = begin_line_heights;
+          mid_line_staff_heights = mid_line_heights;
+        }
+
+      // TODO: consider a pure version of get_grob_direction?
+      Direction d = to_dir (g->get_property_data ("direction"));
+      d = (d == CENTER) ? UP : d;
+
+      Interval_t<int> rank_span = g->spanned_rank_interval ();
+      vsize first_break = lower_bound (ranks, (vsize)rank_span[LEFT], less<vsize> ());
+      if (first_break > 0 && ranks[first_break] >= (vsize)rank_span[LEFT])
+        first_break--;
+
+      for (vsize j = first_break; j + 1 < ranks.size () && (int)ranks[j] <= rank_span[RIGHT]; ++j)
+        {
+          int start = ranks[j];
+          int end = ranks[j + 1];
+
+          // Take grobs that are visible with respect to a slightly longer line.
+          // Otherwise, we will never include grobs at breakpoints which aren't
+          // end-of-line-visible.
+          int visibility_end = j + 2 < ranks.size () ? ranks[j + 2] : end;
+
+          if (g->pure_is_visible (start, visibility_end))
+            {
+              Interval dims = g->pure_height (common, start, end);
+              if (!dims.is_empty ())
+                {
+                  if (rank_span[LEFT] <= start)
+                    {
+                      if (outside_staff)
+                        begin_line_heights[j].unite (begin_line_staff_heights[j].union_disjoint (dims, padding, d));
+                      else
+                        begin_line_heights[j].unite (dims);
+                    }
+                  if (rank_span[RIGHT] > start)
+                    {
+                      if (outside_staff)
+                        mid_line_heights[j].unite (mid_line_staff_heights[j].union_disjoint (dims, padding, d));
+                      else
+                        mid_line_heights[j].unite (dims);
+                    }
+                }
+            }
+        }
+    }
+
+  // Convert begin_line_heights and min_line_heights to SCM.
+  SCM begin_scm = scm_c_make_vector (ranks.size () - 1, SCM_EOL);
+  SCM mid_scm = scm_c_make_vector (ranks.size () - 1, SCM_EOL);
+  for (vsize i = 0; i < begin_line_heights.size (); ++i)
     {
-      int start = Paper_column::get_rank (cols[breaks[i]]);
-      int end = Paper_column::get_rank (cols[breaks[i+1]]);
-      Interval begin_line_iv;
-      Interval mid_line_iv;
-
-      for (vsize j = it_index; j < items.size (); j++)
-       {
-         Item *it = dynamic_cast<Item*> (items[j]);
-         int rank = it->get_column ()->get_rank ();
-
-         if (rank <= end && it->pure_is_visible (start, end)
-             && !to_boolean (it->get_property ("cross-staff")))
-           {
-             Interval dims = items[j]->pure_height (common, start, end);
-             Interval &target_iv = start == it->get_column ()->get_rank () ? begin_line_iv : mid_line_iv;
-
-             if (!dims.is_empty ())
-               target_iv.unite (dims);
-           }
-
-         if (rank < end)
-           it_index++;
-         else if (rank > end)
-           break;
-       }
-
-      for (vsize j = 0; j < spanners.size (); j++)
-       {
-         Interval_t<int> rank_span = spanners[j]->spanned_rank_interval ();
-         if (rank_span[LEFT] <= end && rank_span[RIGHT] >= start
-             && !to_boolean (spanners[j]->get_property ("cross-staff")))
-           {
-             Interval dims = spanners[j]->pure_height (common, start, end);
-
-             if (!dims.is_empty ())
-               mid_line_iv.unite (dims);
-           }
-       }
-
-      scm_vector_set_x (begin_line_heights, scm_from_int (i), ly_interval2scm (begin_line_iv));
-      scm_vector_set_x (mid_line_heights, scm_from_int (i), ly_interval2scm (mid_line_iv));
+      scm_vector_set_x (begin_scm, scm_from_int (i), ly_interval2scm (begin_line_heights[i]));
+      scm_vector_set_x (mid_scm, scm_from_int (i), ly_interval2scm (mid_line_heights[i]));
     }
-  return scm_cons (begin_line_heights, mid_line_heights);
+
+  return scm_cons (begin_scm, mid_scm);
 }
 
 Interval
@@ -199,46 +310,30 @@ Axis_group_interface::relative_pure_height (Grob *me, int start, int end)
      Unfortunately, it isn't always true, particularly if there is a
      VerticalAlignment somewhere in the descendants.
 
-     Apart from PianoStaff, which has a fixed VerticalAlignment so it doesn't
-     count, the only VerticalAlignment comes from Score. This makes it
+     Usually, the only VerticalAlignment comes from Score. This makes it
      reasonably safe to assume that if our parent is a VerticalAlignment,
      we can assume additivity and cache things nicely. */
   Grob *p = me->get_parent (Y_AXIS);
   if (p && Align_interface::has_interface (p))
-    return Axis_group_interface::cached_pure_height (me, start, end);
+    return Axis_group_interface::sum_partial_pure_heights (me, start, end);
 
-  Grob *common = calc_pure_elts_and_common (me);
-  extract_grob_set (me, "pure-relevant-items", items);
-  extract_grob_set (me, "pure-relevant-spanners", spanners);
+  Grob *common = unsmob_grob (me->get_object ("pure-Y-common"));
+  extract_grob_set (me, "pure-relevant-grobs", elts);
 
   Interval r;
-
-  for (vsize i = 0; i < items.size (); i++)
-    {
-      Item *it = dynamic_cast<Item*> (items[i]);
-      int rank = it->get_column ()->get_rank ();
-
-      if (rank > end)
-       break;
-      else if (rank >= start && it->pure_is_visible (start, end)
-              && !to_boolean (it->get_property ("cross-staff")))
-       {
-         Interval dims = it->pure_height (common, start, end);
-         if (!dims.is_empty ())
-           r.unite (dims);
-       }
-    }
-
-  for (vsize i = 0; i < spanners.size (); i++)
+  for (vsize i = 0; i < elts.size (); i++)
     {
-      Interval_t<int> rank_span = spanners[i]->spanned_rank_interval ();
+      Grob *g = elts[i];
+      Interval_t<int> rank_span = g->spanned_rank_interval ();
       if (rank_span[LEFT] <= end && rank_span[RIGHT] >= start
-         && !to_boolean (spanners[i]->get_property ("cross-staff")))
-       {
-         Interval dims = spanners[i]->pure_height (common, start, end);
-         if (!dims.is_empty ())
-           r.unite (dims);
-       }
+          && g->pure_is_visible (start, end)
+          && !(to_boolean (g->get_property ("cross-staff"))
+               && Stem::has_interface (g)))
+        {
+          Interval dims = g->pure_height (common, start, end);
+          if (!dims.is_empty ())
+            r.unite (dims);
+        }
     }
   return r;
 }
@@ -269,13 +364,13 @@ Axis_group_interface::pure_height (SCM smob, SCM start_scm, SCM end_scm)
 
   /* Maybe we are in the second pass of a two-pass spacing run. In that
      case, the Y-extent of a system is already given to us */
-  System *system = dynamic_cast<System*> (me);
+  System *system = dynamic_cast<System *> (me);
   if (system)
     {
       SCM line_break_details = system->column (start)->get_property ("line-break-system-details");
       SCM system_y_extent = scm_assq (ly_symbol2scm ("system-Y-extent"), line_break_details);
       if (scm_is_pair (system_y_extent))
-       return scm_cdr (system_y_extent);
+        return scm_cdr (system_y_extent);
     }
 
   return ly_interval2scm (pure_group_height (me, start, end));
@@ -315,17 +410,17 @@ Axis_group_interface::combine_skylines (SCM smob)
     {
       SCM skyline_scm = elements[i]->get_property ("vertical-skylines");
       if (Skyline_pair::unsmob (skyline_scm))
-       {
-         Real offset = elements[i]->relative_coordinate (y_common, Y_AXIS);
-         Skyline_pair other = *Skyline_pair::unsmob (skyline_scm);
-         other.raise (offset);
-         other.shift (elements[i]->relative_coordinate (x_common, X_AXIS));
-         ret.merge (other);
-       }
+        {
+          Real offset = elements[i]->relative_coordinate (y_common, Y_AXIS);
+          Skyline_pair other = *Skyline_pair::unsmob (skyline_scm);
+          other.raise (offset);
+          other.shift (elements[i]->relative_coordinate (x_common, X_AXIS));
+          ret.merge (other);
+        }
     }
   return ret.smobbed_copy ();
 }
-  
+
 SCM
 Axis_group_interface::generic_group_extent (Grob *me, Axis a)
 {
@@ -349,7 +444,7 @@ Interval
 Axis_group_interface::staff_extent (Grob *me, Grob *refp, Axis ext_a, Grob *staff, Axis parent_a)
 {
   extract_grob_set (me, "elements", elts);
-  vector<Grob*> new_elts;
+  vector<Grob *> new_elts;
 
   for (vsize i = 0; i < elts.size (); i++)
     if (elts[i]->common_refpoint (staff, parent_a) == staff)
@@ -358,57 +453,60 @@ Axis_group_interface::staff_extent (Grob *me, Grob *refp, Axis ext_a, Grob *staf
   return relative_group_extent (new_elts, refp, ext_a);
 }
 
+MAKE_SCHEME_CALLBACK (Axis_group_interface, calc_pure_relevant_grobs, 1);
+SCM
+Axis_group_interface::calc_pure_relevant_grobs (SCM smob)
+{
+  Grob *me = unsmob_grob (smob);
+  return internal_calc_pure_relevant_grobs (me, "elements");
+}
 
-Grob *
-Axis_group_interface::calc_pure_elts_and_common (Grob *me)
+SCM
+Axis_group_interface::internal_calc_pure_relevant_grobs (Grob *me, string grob_set_name)
 {
-  if (Grob *c = unsmob_grob (me->get_object ("pure-Y-common")))
-    return c;
-  
-  extract_grob_set (me, "elements", elts);
+  extract_grob_set (me, grob_set_name.c_str (), elts);
 
-  vector<Grob*> relevant_items;
-  vector<Grob*> relevant_spanners;
+  vector<Grob *> relevant_grobs;
   SCM pure_relevant_p = ly_lily_module_constant ("pure-relevant?");
 
   for (vsize i = 0; i < elts.size (); i++)
     {
       if (to_boolean (scm_apply_1 (pure_relevant_p, elts[i]->self_scm (), SCM_EOL)))
-       {
-         if (dynamic_cast<Item*> (elts[i]))
-           relevant_items.push_back (elts[i]);
-         else if (dynamic_cast<Spanner*> (elts[i]))
-           relevant_spanners.push_back (elts[i]);
-       }
-           
-
-      Item *it = dynamic_cast<Item*> (elts[i]);
-      Direction d = LEFT;
-      if (it)
-       do
-         {
-           Item *piece = it->find_prebroken_piece (d);
-           if (piece && to_boolean (scm_apply_1 (pure_relevant_p, piece->self_scm (), SCM_EOL)))
-             relevant_items.push_back (piece);
-         }
-       while (flip (&d) != LEFT);
+        relevant_grobs.push_back (elts[i]);
+
+      if (Item *it = dynamic_cast<Item *> (elts[i]))
+        {
+          for (LEFT_and_RIGHT (d))
+            {
+              Item *piece = it->find_prebroken_piece (d);
+              if (piece && to_boolean (scm_apply_1 (pure_relevant_p, piece->self_scm (), SCM_EOL)))
+                relevant_grobs.push_back (piece);
+            }
+        }
     }
-  vector_sort (relevant_items, Item::less);
 
-  Grob *common = common_refpoint_of_array (relevant_items, me, Y_AXIS);
-  common = common_refpoint_of_array (relevant_spanners, common, Y_AXIS);
+  vector_sort (relevant_grobs, pure_staff_priority_less);
+  SCM grobs_scm = Grob_array::make_array ();
+  unsmob_grob_array (grobs_scm)->set_array (relevant_grobs);
 
-  me->set_object ("pure-Y-common", common->self_scm ());
-  
-  SCM items_scm = Grob_array::make_array ();
-  SCM spanners_scm = Grob_array::make_array ();
+  return grobs_scm;
+}
 
-  unsmob_grob_array (items_scm)->set_array (relevant_items);
-  unsmob_grob_array (spanners_scm)->set_array (relevant_spanners);
-  me->set_object ("pure-relevant-items", items_scm);
-  me->set_object ("pure-relevant-spanners", spanners_scm);
+MAKE_SCHEME_CALLBACK (Axis_group_interface, calc_pure_y_common, 1);
+SCM
+Axis_group_interface::calc_pure_y_common (SCM smob)
+{
+  Grob *me = unsmob_grob (smob);
+
+  extract_grob_set (me, "pure-relevant-grobs", elts);
+  Grob *common = common_refpoint_of_array (elts, me, Y_AXIS);
+  if (!common)
+    {
+      me->programming_error ("No common parent found in calc_pure_y_common.");
+      return SCM_EOL;
+    }
 
-  return common;
+  return common->self_scm ();
 }
 
 SCM
@@ -421,11 +519,10 @@ Axis_group_interface::calc_common (Grob *me, Axis axis)
       me->programming_error ("No common parent found in calc_common axis.");
       return SCM_EOL;
     }
-  
+
   return common->self_scm ();
 }
 
-
 MAKE_SCHEME_CALLBACK (Axis_group_interface, calc_x_common, 1);
 SCM
 Axis_group_interface::calc_x_common (SCM grob)
@@ -443,8 +540,13 @@ Axis_group_interface::calc_y_common (SCM grob)
 Interval
 Axis_group_interface::pure_group_height (Grob *me, int start, int end)
 {
-  Grob *common = calc_pure_elts_and_common (me);
-       
+  Grob *common = unsmob_grob (me->get_object ("pure-Y-common"));
+
+  if (!common)
+    {
+      programming_error ("no pure Y common refpoint");
+      return Interval ();
+    }
   Real my_coord = me->relative_coordinate (common, Y_AXIS);
   Interval r (relative_pure_height (me, start, end));
 
@@ -452,7 +554,7 @@ Axis_group_interface::pure_group_height (Grob *me, int start, int end)
 }
 
 void
-Axis_group_interface::get_children (Grob *me, vector<Grob*> *found)
+Axis_group_interface::get_children (Grob *me, vector<Grob *> *found)
 {
   found->push_back (me);
 
@@ -467,8 +569,8 @@ Axis_group_interface::get_children (Grob *me, vector<Grob*> *found)
     }
 }
 
-bool
-staff_priority_less (Grob * const &g1, Grob * const &g2)
+static bool
+staff_priority_less (Grob *const &g1, Grob *const &g2)
 {
   Real priority_1 = robust_scm2double (g1->get_property ("outside-staff-priority"), -infinity_f);
   Real priority_2 = robust_scm2double (g2->get_property ("outside-staff-priority"), -infinity_f);
@@ -491,6 +593,15 @@ staff_priority_less (Grob * const &g1, Grob * const &g2)
   return start_1 < start_2;
 }
 
+static bool
+pure_staff_priority_less (Grob *const &g1, Grob *const &g2)
+{
+  Real priority_1 = robust_scm2double (g1->get_property ("outside-staff-priority"), -infinity_f);
+  Real priority_2 = robust_scm2double (g2->get_property ("outside-staff-priority"), -infinity_f);
+
+  return priority_1 < priority_2;
+}
+
 static void
 add_boxes (Grob *me, Grob *x_common, Grob *y_common, vector<Box> *const boxes, Skyline_pair *skylines)
 {
@@ -505,13 +616,13 @@ add_boxes (Grob *me, Grob *x_common, Grob *y_common, vector<Box> *const boxes, S
   else if (Grob_array *elements = unsmob_grob_array (me->get_object ("elements")))
     {
       for (vsize i = 0; i < elements->size (); i++)
-       add_boxes (elements->grob (i), x_common, y_common, boxes, skylines);
+        add_boxes (elements->grob (i), x_common, y_common, boxes, skylines);
     }
   else if (!scm_is_number (me->get_property ("outside-staff-priority"))
-          && !to_boolean (me->get_property ("cross-staff")))
+           && !to_boolean (me->get_property ("cross-staff")))
     {
       boxes->push_back (Box (me->extent (x_common, X_AXIS),
-                            me->extent (y_common, Y_AXIS)));
+                             me->extent (y_common, Y_AXIS)));
     }
 }
 
@@ -536,9 +647,9 @@ add_boxes (Grob *me, Grob *x_common, Grob *y_common, vector<Box> *const boxes, S
 */
 static void
 add_grobs_of_one_priority (Skyline_pair *const skylines,
-                          vector<Grob*> elements,
-                          Grob *x_common,
-                          Grob *y_common)
+                           vector<Grob *> elements,
+                           Grob *x_common,
+                           Grob *y_common)
 {
   vector<Box> boxes;
   Drul_array<Real> last_affected_position;
@@ -550,48 +661,57 @@ add_grobs_of_one_priority (Skyline_pair *const skylines,
       last_affected_position[DOWN] = -infinity_f;
       /* do one pass */
       for (vsize i = elements.size (); i--;)
-       {
-         Direction dir = get_grob_direction (elements[i]);
-         if (dir == CENTER)
-           {
-             warning (_ ("an outside-staff object should have a direction, defaulting to up"));
-             dir = UP;
-           }
-
-         Box b (elements[i]->extent (x_common, X_AXIS),
-                elements[i]->extent (y_common, Y_AXIS));
-         SCM horizon_padding_scm = elements[i]->get_property ("outside-staff-horizontal-padding");
-         Real horizon_padding = robust_scm2double (horizon_padding_scm, 0.0);
-
-         if (b[X_AXIS][LEFT] - 2*horizon_padding < last_affected_position[dir])
-           continue;
-
-         if (!b[X_AXIS].is_empty () && !b[Y_AXIS].is_empty ())
-           {
-             boxes.clear ();
-             boxes.push_back (b);
-             Skyline other = Skyline (boxes, horizon_padding, X_AXIS, -dir);
-             Real padding = robust_scm2double (elements[i]->get_property ("outside-staff-padding"), 0.5);
-             Real dist = (*skylines)[dir].distance (other) + padding;
-
-             if (dist > 0)
-               {
-                 b.translate (Offset (0, dir*dist));
-                 elements[i]->translate_axis (dir*dist, Y_AXIS);
-               }
-             (*skylines)[dir].insert (b, 0, X_AXIS);
-             elements[i]->set_property ("outside-staff-priority", SCM_BOOL_F);
-             last_affected_position[dir] = b[X_AXIS][RIGHT];
-           }
-
-         /*
-           Ugh: quadratic. --hwn
-          */
-         elements.erase (elements.begin () + i);
-       }
+        {
+          Direction dir = get_grob_direction (elements[i]);
+          if (dir == CENTER)
+            {
+              warning (_ ("an outside-staff object should have a direction, defaulting to up"));
+              dir = UP;
+            }
+
+          Box b (elements[i]->extent (x_common, X_AXIS),
+                 elements[i]->extent (y_common, Y_AXIS));
+          SCM horizon_padding_scm = elements[i]->get_property ("outside-staff-horizontal-padding");
+          Real horizon_padding = robust_scm2double (horizon_padding_scm, 0.0);
+
+          if (b[X_AXIS][LEFT] - 2 * horizon_padding < last_affected_position[dir])
+            continue;
+
+          if (!b[X_AXIS].is_empty () && !b[Y_AXIS].is_empty ())
+            {
+              boxes.clear ();
+              boxes.push_back (b);
+              Skyline other = Skyline (boxes, horizon_padding, X_AXIS, -dir);
+              Real padding = robust_scm2double (elements[i]->get_property ("outside-staff-padding"), 0.5);
+              Real dist = (*skylines)[dir].distance (other) + padding;
+
+              if (dist > 0)
+                {
+                  b.translate (Offset (0, dir * dist));
+                  elements[i]->translate_axis (dir * dist, Y_AXIS);
+                }
+              skylines->insert (b, 0, X_AXIS);
+              elements[i]->set_property ("outside-staff-priority", SCM_BOOL_F);
+              last_affected_position[dir] = b[X_AXIS][RIGHT];
+            }
+
+          /*
+            Ugh: quadratic. --hwn
+           */
+          elements.erase (elements.begin () + i);
+        }
     }
 }
 
+bool
+Axis_group_interface::has_outside_staff_parent (Grob *me)
+{
+  return (me
+          ? (scm_is_number (me->get_property ("outside-staff-priority"))
+             || has_outside_staff_parent (me->get_parent (Y_AXIS)))
+          : false);
+}
+
 // TODO: it is tricky to correctly handle skyline placement of cross-staff grobs.
 // For example, cross-staff beams cannot be formatted until the distance between
 // staves is known and therefore any grobs that depend on the beam cannot be placed
@@ -602,7 +722,7 @@ add_grobs_of_one_priority (Skyline_pair *const skylines,
 // that there is no room for the cross-staff grob. It also means, of course, that
 // we don't get the benefits of skyline placement for cross-staff grobs.
 Skyline_pair
-Axis_group_interface::skyline_spacing (Grob *me, vector<Grob*> elements)
+Axis_group_interface::skyline_spacing (Grob *me, vector<Grob *> elements)
 {
   /* For grobs with an outside-staff-priority, the sorting function might
      call extent and cause suicide. This breaks the contract that is required
@@ -624,8 +744,8 @@ Axis_group_interface::skyline_spacing (Grob *me, vector<Grob*> elements)
 
   Skyline_pair skylines;
   for (i = 0; i < elements.size ()
-        && !scm_is_number (elements[i]->get_property ("outside-staff-priority")); i++)
-    if (!to_boolean (elements[i]->get_property ("cross-staff")))
+       && !scm_is_number (elements[i]->get_property ("outside-staff-priority")); i++)
+    if (!(to_boolean (elements[i]->get_property ("cross-staff")) || has_outside_staff_parent (elements[i])))
       add_boxes (elements[i], x_common, y_common, &boxes, &skylines);
 
   SCM padding_scm = me->get_property ("skyline-horizontal-padding");
@@ -634,18 +754,18 @@ Axis_group_interface::skyline_spacing (Grob *me, vector<Grob*> elements)
   for (; i < elements.size (); i++)
     {
       if (to_boolean (elements[i]->get_property ("cross-staff")))
-       continue;
+        continue;
 
       SCM priority = elements[i]->get_property ("outside-staff-priority");
-      vector<Grob*> current_elts;
+      vector<Grob *> current_elts;
       current_elts.push_back (elements[i]);
-      while (i + 1 < elements.size () 
-            && scm_eq_p (elements[i+1]->get_property ("outside-staff-priority"), priority))
-       {
-         if (!to_boolean (elements[i+1]->get_property ("cross-staff")))
-           current_elts.push_back (elements[i+1]);
-         ++i;
-       }
+      while (i + 1 < elements.size ()
+             && scm_is_eq (elements[i + 1]->get_property ("outside-staff-priority"), priority))
+        {
+          if (!to_boolean (elements[i + 1]->get_property ("cross-staff")))
+            current_elts.push_back (elements[i + 1]);
+          ++i;
+        }
 
       add_grobs_of_one_priority (&skylines, current_elts, x_common, y_common);
     }
@@ -653,21 +773,6 @@ Axis_group_interface::skyline_spacing (Grob *me, vector<Grob*> elements)
   return skylines;
 }
 
-MAKE_SCHEME_CALLBACK (Axis_group_interface, calc_max_stretch, 1)
-SCM
-Axis_group_interface::calc_max_stretch (SCM smob)
-{
-  Grob *me = unsmob_grob (smob);
-  Real ret = 0;
-  extract_grob_set (me, "elements", elts);
-
-  for (vsize i = 0; i < elts.size (); i++)
-    if (Axis_group_interface::has_interface (elts[i]))
-      ret += robust_scm2double (elts[i]->get_property ("max-stretch"), 0.0);
-
-  return scm_from_double (ret);
-}
-
 MAKE_SCHEME_CALLBACK (Axis_group_interface, print, 1)
 SCM
 Axis_group_interface::print (SCM smob)
@@ -680,27 +785,87 @@ Axis_group_interface::print (SCM smob)
   if (Skyline_pair *s = Skyline_pair::unsmob (me->get_property ("vertical-skylines")))
     {
       ret.add_stencil (Lookup::points_to_line_stencil (0.1, (*s)[UP].to_points (X_AXIS))
-                      .in_color (255, 0, 255));
+                       .in_color (255, 0, 255));
       ret.add_stencil (Lookup::points_to_line_stencil (0.1, (*s)[DOWN].to_points (X_AXIS))
-                      .in_color (0, 255, 255));
+                       .in_color (0, 255, 255));
     }
   return ret.smobbed_copy ();
 }
 
+MAKE_SCHEME_CALLBACK (Axis_group_interface, calc_pure_staff_staff_spacing, 3)
+SCM
+Axis_group_interface::calc_pure_staff_staff_spacing (SCM smob, SCM start, SCM end)
+{
+  return calc_maybe_pure_staff_staff_spacing (unsmob_grob (smob),
+                                              true,
+                                              scm_to_int (start),
+                                              scm_to_int (end));
+}
+
+MAKE_SCHEME_CALLBACK (Axis_group_interface, calc_staff_staff_spacing, 1)
+SCM
+Axis_group_interface::calc_staff_staff_spacing (SCM smob)
+{
+  return calc_maybe_pure_staff_staff_spacing (unsmob_grob (smob),
+                                              false,
+                                              0,
+                                              INT_MAX);
+}
+
+SCM
+Axis_group_interface::calc_maybe_pure_staff_staff_spacing (Grob *me, bool pure, int start, int end)
+{
+  Grob *grouper = unsmob_grob (me->get_object ("staff-grouper"));
+
+  if (grouper)
+    {
+      bool within_group = Staff_grouper_interface::maybe_pure_within_group (grouper, me, pure, start, end);
+      if (within_group)
+        return grouper->get_maybe_pure_property ("staff-staff-spacing", pure, start, end);
+      else
+        return grouper->get_maybe_pure_property ("staffgroup-staff-spacing", pure, start, end);
+    }
+  return me->get_maybe_pure_property ("default-staff-staff-spacing", pure, start, end);
+}
+
+Real
+Axis_group_interface::minimum_distance (Grob *g1, Grob *g2, Axis a)
+{
+  SCM sym = ly_symbol2scm ((a == Y_AXIS) ? "vertical-skylines" : "horizontal-skylines");
+
+  Skyline_pair *s1 = Skyline_pair::unsmob (g1->get_property (sym));
+  Skyline_pair *s2 = Skyline_pair::unsmob (g2->get_property (sym));
+  if (s1 && s2)
+    return (*s1)[DOWN].distance ((*s2)[UP]);
+  return 0;
+}
+
 ADD_INTERFACE (Axis_group_interface,
-              "An object that groups other layout objects.",
-
-              /* properties */
-              "X-common "
-              "Y-common "
-              "adjacent-pure-heights "
-              "axes "
-              "elements "
-              "keep-fixed-while-stretching "
-              "max-stretch "
-              "no-alignment "
-              "pure-Y-common "
-              "pure-relevant-items "
-              "pure-relevant-spanners "
-              "vertical-skylines "
-              );
+               "An object that groups other layout objects.",
+
+               // TODO: some of these properties are specific to
+               // VerticalAxisGroup. We should split off a
+               // vertical-axis-group-interface.
+               /* properties */
+               "adjacent-pure-heights "
+               "axes "
+               "bound-alignment-interfaces "
+               "default-staff-staff-spacing "
+               "elements "
+               "max-stretch "
+               "no-alignment "
+               "nonstaff-nonstaff-spacing "
+               "nonstaff-relatedstaff-spacing "
+               "nonstaff-unrelatedstaff-spacing "
+               "pure-relevant-grobs "
+               "pure-relevant-items "
+               "pure-relevant-spanners "
+               "pure-Y-common "
+               "staff-affinity "
+               "staff-grouper "
+               "staff-staff-spacing "
+               "system-Y-offset "
+               "vertical-skylines "
+               "X-common "
+               "Y-common "
+              );