LedgerLineSpanner included in vertical skylines of VerticalAxisGroup.

An elements grob array was erroneously being created for the
LedgerLinesSpanner. This has been eliminated, which now means that its
stencil can be used to calculate its vertical skylines.

diff --git a/input/regression/ledger-lines-dynamics.ly b/input/regression/ledger-lines-dynamics.ly
new file mode 100644 (file)
index 0000000..a6e2d80
--- /dev/null
+++ b/input/regression/ledger-lines-dynamics.ly
@@ -0,0 +1,10 @@
+\version "2.17.2"
+
+\header {
+  texidoc = "Dynamics and other outside staff objects avoid ledger lines.
+"
+}
+
+\relative c' {
+  f16\pp[ c d e ] r2. |
+}

diff --git a/lily/ledger-line-engraver.cc b/lily/ledger-line-engraver.cc
index 614874a8e83fa74c7220cbb097458160e7552220..2bb12c9cdcbb7c0c7e6d0ca3c696d0e47ac5b745 100644 (file)
--- a/lily/ledger-line-engraver.cc
+++ b/lily/ledger-line-engraver.cc
@@ -95,7 +95,6 @@ Ledger_line_engraver::stop_spanner ()
   if (span_)
     {
       span_->set_bound (RIGHT, unsmob_grob (get_property ("currentCommandColumn")));
-      Pointer_group_interface::set_ordered (span_, ly_symbol2scm ("elements"), false);
       span_ = 0;
     }
 }

diff --git a/scm/define-grobs.scm b/scm/define-grobs.scm
index 5083b5053d6220d30224515054c8090a2dfc6a90..528fdda26b000bbe20e3ffb2bd8603ba78d343c1 100644 (file)
--- a/scm/define-grobs.scm
+++ b/scm/define-grobs.scm
@@ -1206,6 +1206,7 @@
        (minimum-length-fraction . 0.25)
        (springs-and-rods . ,ly:ledger-line-spanner::set-spacing-rods)
        (stencil . ,ly:ledger-line-spanner::print)
+       (vertical-skylines . ,ly:grob::vertical-skylines-from-stencil)
        (X-extent . #f)
        (Y-extent . #f)
        (meta . ((class . Spanner)