Re: Uses horizontal skylines in accidental placement (issue 6489086)

[Janek Warchoł]

On Fri, Sep 28, 2012 at 8:56 AM,  <address@hidden> wrote:
> http://codereview.appspot.com/6489086/diff/11/lily/accidental-placement.cc#newcode377
> lily/accidental-placement.cc:377: Real offset =
> -ape->horizontal_skylines_[RIGHT].distance (left_skyline);
> Both Mike and Janek thought the accidentals needed a bit more padding
> for close intervals, so I think Janek should try horizon_padding here
> with  distance(left_skyline, 0.1)

>
> http://codereview.appspot.com/6489086/diff/11/lily/accidental-placement.cc
> File lily/accidental-placement.cc (right):
>
I apologize for not replying earlier.  I've tried horizon_padding and
i like the results; see more information in tracker issue
http://code.google.com/p/lilypond/issues/detail?id=2141#c16

thanks,
Janek