Hey all,
My patch did change output (it prevented duplicate ledger lines), although I
did not spot the result that Janek sees.
The old patch is also flawed, in that it only changes the ledger line directly
next to the accidental. The entire bounding box of the accidental should,
however, be taken into account.
I maintain that the best solution is:
{
\override Staff.Accidental #'layer = #-100
\override Staff.LedgerLineSpanner #'layer = #-101
\override Staff.Accidental #'whiteout = ##t
<des ces'>
}
Which is a trivial change to engravers-init.ly. I'll post a patch later today
unless anyone sees a better alternative.
Cheers,
MS
On Aug 22, 2011, at 11:33 PM, Han-Wen Nienhuys wrote:
> 2011/8/22 Janek Warchoł <janek.lilyp...@gmail.com>:
>> I wholeheartedly disagree, i think that this issue isn't negligible
>> (or did i misunderstood you, Han-Wen?).
>
> My impression is that the patch was not actually changing output.
>
> If it was intended to, I couldn't tell from a read of the patch.
>
> --
> Han-Wen Nienhuys - han...@xs4all.nl - http://www.xs4all.nl/~hanwen
>
> _______________________________________________
> lilypond-devel mailing list
> lilypond-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/lilypond-devel
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
