On 20 avr. 2013, at 08:06, k-ohara5...@oco.net wrote:

> 
> https://codereview.appspot.com/8819043/diff/5001/lily/line-spanner.cc
> File lily/line-spanner.cc (right):
> 
> https://codereview.appspot.com/8819043/diff/5001/lily/line-spanner.cc#newcode371
> lily/line-spanner.cc:371: // actual stencil bounds may be different
> The actual stencil bounds are (inf , -inf) if someone sets style=none
> When I try this on input/regression/dynamics-alignment-no-line.ly the
> while loop in add_grobs_of_one_priority() takes forever.
> 
> https://codereview.appspot.com/8819043/diff/5001/lily/line-spanner.cc#newcode377
> lily/line-spanner.cc:377: Real new_left_y = minmax (-d, extremity,
> line_left[Y_AXIS] + d * skyline_spanner_padding);
> The smaller of the extent of the cresc.--- and the line padded by
> skyline_spanner_padding.  Is that what you want?
> 

Yeah, because we never want the padding going outside of the grob.

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

Reply via email to