[Top][All Lists]
[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;
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Implement thick bar lines (use \bar "." for this),
hanwenn <=