lilypond-user
[Top][All Lists]
Advanced

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

Re: [PATCH] Added chromatic ledger and internal ledger lines. Sample tes


From: Kevin Dalley
Subject: Re: [PATCH] Added chromatic ledger and internal ledger lines. Sample test file is
Date: Sat, 24 Mar 2007 18:38:42 -0700
User-agent: Gnus/5.110006 (No Gnus v0.6) XEmacs/21.4.19 (linux)

One more patch for the chromatic staff.  I reverted this change.  My
original version does not work at all when a ledger line is at
position 0.  This will do better, but the ledger line code still needs
some revisions for this case.

Please tell me if you would like a patch which starts from the current
2.11.  I'm willing to give it a try, if I can figure out enough git.

>From b530144219ff01bb837b559859f5a16ab5548709 Mon Sep 17 00:00:00 2001
From: Kevin Dalley <address@hidden>
Date: Sat, 24 Mar 2007 18:33:55 -0700
Subject: [PATCH] Ignore pos if pos is 0.
Restores original behavior, though this should be fixed to allow for ledger 
lines
at position 0.
---
 lily/ledger-line-spanner.cc |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/lily/ledger-line-spanner.cc b/lily/ledger-line-spanner.cc
index 8401d2d..1ec9a68 100644
--- a/lily/ledger-line-spanner.cc
+++ b/lily/ledger-line-spanner.cc
@@ -526,7 +526,7 @@ Ledger_line_spanner::print (SCM smob)
       Item *h = dynamic_cast<Item *> (heads[i]);
 
       int pos = Staff_symbol_referencer::get_rounded_position (h);
-      if (!staff_extent.contains (pos - sign (pos)))
+      if (pos && !staff_extent.contains (pos - sign (pos)))
        {
          Interval ledger_size;
          Real left_shorten;
-- 
1.5.0.3





reply via email to

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