[Top][All Lists]
[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:39:13 -0700 |
User-agent: |
Gnus/5.110006 (No Gnus v0.6) XEmacs/21.4.19 (linux) |
Oops. This was supposed to be for lilypond-devel.
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