From: Neil Puttock <n.puttock@gmail.com>
Date: Sat, 18 Jun 2011 22:51:56 +0000 (+0100)
Subject: Fix #1696: Bad breathing sign Y-offset when using line-positions.
X-Git-Tag: release/2.14.2-1~49
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=3752ba5b6d7f811e7ec9419277a07b1de01d9943;p=lilypond.git

Fix #1696: Bad breathing sign Y-offset when using line-positions.

* input/regression/breathing-sign-custom-staff.ly

  new regtest

* lily/breathing-sign.cc (offset_callback):

  use Staff_symbol::line_span () to calculate offset instead of
  Staff_symbol_referencer::line_count (): the latter is only useful for
  symmetrical staves

  tidy code; remove obsolete TODO
---

diff --git a/input/regression/breathing-sign-custom-staff.ly b/input/regression/breathing-sign-custom-staff.ly
new file mode 100644
index 0000000000..98305b0510
--- /dev/null
+++ b/input/regression/breathing-sign-custom-staff.ly
@@ -0,0 +1,13 @@
+\version "2.15.2"
+
+\header {
+  texidoc = "Breathing signs are positioned correctly on custom staves
+which use @code{line-positions}."
+}
+
+\relative c' {
+  \override Staff.StaffSymbol #'line-positions = #'(-7 -5 -3 -1)
+  b2 \breathe b
+  \override BreathingSign #'direction = #DOWN
+  \breathe
+}
diff --git a/lily/breathing-sign.cc b/lily/breathing-sign.cc
index 09fd1d7c51..a59bd65070 100644
--- a/lily/breathing-sign.cc
+++ b/lily/breathing-sign.cc
@@ -20,20 +20,16 @@
 
 #include "breathing-sign.hh"
 
-#include "staff-symbol-referencer.hh"
-#include "directional-element-interface.hh"
-#include "output-def.hh"
-#include "lookup.hh"
 #include "dimensions.hh"
 #include "direction.hh"
-#include "text-interface.hh"
+#include "directional-element-interface.hh"
 #include "font-interface.hh"
 #include "grob.hh"
-
-/*
-  TODO: thickness should be a grob property (unit: linethickness)
-  rather than hardwired to (staff_space / 6).
-*/
+#include "lookup.hh"
+#include "output-def.hh"
+#include "staff-symbol.hh"
+#include "staff-symbol-referencer.hh"
+#include "text-interface.hh"
 
 /*
   UGH : this is full of C&P code. Consolidate!  --hwn
@@ -177,14 +173,20 @@ Breathing_sign::offset_callback (SCM smob)
       set_grob_direction (me, d);
     }
 
-  Real inter = Staff_symbol_referencer::staff_space (me) / 2;
-  int sz = Staff_symbol_referencer::line_count (me) - 1;
-  return scm_from_double (inter * sz * d);
+  Grob *staff = Staff_symbol_referencer::get_staff_symbol (me);
+  if (staff)
+    {
+      Interval iv = Staff_symbol::line_span (staff);
+      Real inter = Staff_symbol::staff_space (me) / 2;
+      return scm_from_double (inter * iv[d]);
+    }
+
+  return scm_from_double (0.0);
 }
 
 ADD_INTERFACE (Breathing_sign,
 	       "A breathing sign.",
-	       
+
 	       /* properties */
 	       "direction "
 	       );