[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Makes sure that ledger lines do not overlap with accidentals. (issue
From: |
Han-Wen Nienhuys |
Subject: |
Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060) |
Date: |
Thu, 18 Aug 2011 09:51:08 -0300 |
On Thu, Aug 18, 2011 at 9:27 AM, <address@hidden> wrote:
> I figured not making a regtest just cuz there will already be little
> changes to several regtests. But, I can certainly add one.
>
> Cheers,
> MS
>
>
> http://codereview.appspot.com/4898060/diff/1/lily/ledger-line-spanner.cc
> File lily/ledger-line-spanner.cc (right):
>
> http://codereview.appspot.com/4898060/diff/1/lily/ledger-line-spanner.cc#newcode89
> lily/ledger-line-spanner.cc:89: continue;
> On 2011/08/18 12:16:50, hanwenn wrote:
>>
>> can't you check poss.find(max(0, lincount - *it)) ?
>
>> I don't understand what this is supposed to do ; in which cases do you
>
> want to
>>
>> skip ledger creation?
>
> Good call - I can do this.
>
> It skips over all ledger lines that already have a note on them, thus
> avoiding duplicates.
are you sure this works with chords that have seconds in them? It
sounds like an optimization that doesn't really improve output at all,
and performance not by much. Why not skip it altogether and avoid the
complexity?
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), hanwenn, 2011/08/18
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), mtsolo, 2011/08/18
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060),
Han-Wen Nienhuys <=
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Mike Solomon, 2011/08/18
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Janek Warchoł, 2011/08/22
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Han-Wen Nienhuys, 2011/08/22
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Mike Solomon, 2011/08/23
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Han-Wen Nienhuys, 2011/08/23
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Mike Solomon, 2011/08/23
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Han-Wen Nienhuys, 2011/08/23
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Jan Nieuwenhuizen, 2011/08/23
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Janek Warchoł, 2011/08/27
- Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060), Jan Nieuwenhuizen, 2011/08/27