From: Kevin Dalley <kevin@kelphead.org>
Date: Sun, 18 Mar 2007 16:30:57 +0000 (-0700)
Subject: Draw bar lines correctly when staff is not centered at zero.
X-Git-Tag: release/2.11.52-1~10^2~2
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=173e30f6781380bb3eeb1640ac309585712900ef;p=lilypond.git

Draw bar lines correctly when staff is not centered at zero.

Signed-off-by: Neil Puttock <n.puttock@gmail.com>
---

diff --git a/input/regression/non-centered-bar-lines.ly b/input/regression/non-centered-bar-lines.ly
new file mode 100644
index 0000000000..3c0738c96c
--- /dev/null
+++ b/input/regression/non-centered-bar-lines.ly
@@ -0,0 +1,12 @@
+\version "2.11.52"
+\header {
+  texidoc = "Bar lines are positioned correctly when using custom
+staves which are not centered around position@tie{}0.
+"
+}
+\new Staff {
+  \override Staff.StaffSymbol #'line-positions = #'(1 3 5 7 9)
+  c''1 \bar "||"
+  c''1 \bar ":"
+  c''1 \bar "|."
+}
diff --git a/lily/bar-line.cc b/lily/bar-line.cc
index e51aad7196..4d2cbf9b59 100644
--- a/lily/bar-line.cc
+++ b/lily/bar-line.cc
@@ -94,13 +94,28 @@ Bar_line::compound_barline (Grob *me, string str, Real h,
   colon.translate_axis (-dist / 2, Y_AXIS);
 
   Stencil m;
+  Grob *staff = Staff_symbol_referencer::get_staff_symbol (me);
+  Real center = 0;
+  if (staff)
+    {
+      Interval staff_extent = staff->extent (staff, Y_AXIS);
+      center = staff_extent.center ();
+    }
+
   if (str == "||:")
     str = "|:";
 
   if (str == "")
-    return Lookup::blank (Box (Interval (0, 0), Interval (-h / 2, h / 2)));
+    {
+      Stencil empty =  Lookup::blank (Box (Interval (0, 0), Interval (-h / 2, h / 2)));
+      empty.translate_axis (center, Y_AXIS);
+      return empty;
+    }
   else if (str == "|")
-    return thin;
+    {
+      thin.translate_axis (center, Y_AXIS);
+      return thin;
+    }
   else if (str == "|." || (h == 0 && str == ":|"))
     {
       m.add_at_edge (X_AXIS, LEFT, thick, 0);
@@ -165,6 +180,8 @@ Bar_line::compound_barline (Grob *me, string str, Real h,
     {
       m = dot;
     }
+
+  m.translate_axis (center, Y_AXIS);
   return m;
 }