On 12 avr. 2013, at 22:29, k-ohara5...@oco.net wrote:

> 
> https://codereview.appspot.com/7310075/diff/67001/lily/skyline.cc
> File lily/skyline.cc (right):
> 
> https://codereview.appspot.com/7310075/diff/67001/lily/skyline.cc#newcode370
> lily/skyline.cc:370: if (x1 >= last_end)
> Oops.  This controls whether to put an empty, -inf height, building in
> the gap. (When do we ever want Building data structures marking gaps?
> The skyline concept seems to allow for gaps.)
> 
> These zero-width anti-buildings would not have any effect, except that
> they are drawn (issue 3311) but they are not the boxes that surprised us
> when they disappeared (issue 3161).
> 
> Probably safest to put back the x1 > last_end + EPS
> 
> https://codereview.appspot.com/7310075/

Fair 'nuf. Can you write a patch?

Cheers,
MS
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to