From: Mike Solomon <mike@apollinemike.com>
Date: Tue, 4 Sep 2012 22:52:19 +0000 (+0200)
Subject: LedgerLineSpanner included in vertical skylines of VerticalAxisGroup.
X-Git-Tag: release/2.17.2-1~16
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=11a21e9285f06006e10ca84ba6d7319e72d38d85;p=lilypond.git

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
index 0000000000..a6e2d80c4c
--- /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 614874a8e8..2bb12c9cdc 100644
--- 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 5083b5053d..528fdda26b 100644
--- 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)