Status: Accepted
Owner: ----
CC: [email protected]
Labels: Type-Critical Regression

New issue 3311 by [email protected]: slur skylines look weird after we allowed zero-width buildings
http://code.google.com/p/lilypond/issues/detail?id=3311

#(ly:set-option 'debug-skylines #t)
{ d''\f( f'') }

see attached.
The change is introduced by this commit:

commit 6f13ef4a292625c1a9d528ac88aec98071ba6388
Author: Mike Solomon <[email protected]>
Date:   Fri Mar 22 07:29:57 2013 +0100

    Permits all positive and zero-length buildings in skylines (issue 3161)

    This allows point stencils to figure into vertical spacing, as shown
    in the new regtest input/regression/skyline-point-extent.ly.

    Some issues regarding floating point errors were raised while reviewing
    this patch, but it was concluded that the math in this patch likely does
    not risk triggering those errors with the way GCC compiles the code.

Maybe this change isn't a problem, but it definitely looks wrong, so i'm setting it to Critical. Feel free to downgrade.

Attachments:
        weird skylines.png  29.5 KB

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

Reply via email to