Re: Makes sure that ledger lines do not overlap with accidentals. (issue 4898060)

[Mike Solomon]

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ł <address@hidden>:
>> 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 - address@hidden - http://www.xs4all.nl/~hanwen
> 
> _______________________________________________
> lilypond-devel mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/lilypond-devel