lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Implement thick bar lines (use \bar "." for this)


From: hanwenn
Subject: Implement thick bar lines (use \bar "." for this)
Date: Tue, 23 Dec 2008 13:46:50 +0000

Reviewers: Reinhold,

Message:
LGTM

missing documentation, and regtest though

Description:
Implement thick bar lines (use \bar "." for this)
Until now, \bar"." produced a single dot instead of the bar line, which
was however not used anywhere and also not documented...
Using "." for the bar line style is consistent with other types like
".|" or "|.", where a | indicates a think line and . indicates a thick
line.

Please review this at http://codereview.appspot.com/11044

Affected files:
  M lily/bar-line.cc


Index: lily/bar-line.cc
diff --git a/lily/bar-line.cc b/lily/bar-line.cc
index df97638dba6b9faa9f43e95d1780454e35d0a5c7..0013b189615564142e768aeda0cc9127e72ae011 100644
--- a/lily/bar-line.cc
+++ b/lily/bar-line.cc
@@ -116,6 +116,11 @@ Bar_line::compound_barline (Grob *me, string str, Real h,
       thin.translate_axis (center, Y_AXIS);
       return thin;
     }
+  else if (str == ".")
+    {
+      thick.translate_axis (center, Y_AXIS);
+      return thick;
+    }
   else if (str == "|." || (h == 0 && str == ":|"))
     {
       m.add_at_edge (X_AXIS, LEFT, thick, 0);
@@ -198,10 +203,6 @@ Bar_line::compound_barline (Grob *me, string str, Real h,
     {
       m = dashed_bar_line (me, h, hair);
     }
-  else if (str == ".")
-    {
-      m = dot;
-    }

   m.translate_axis (center, Y_AXIS);
   return m;






reply via email to

[Prev in Thread] Current Thread [Next in Thread]