"m...@mikesolomon.org" <m...@mikesolomon.org> writes: > 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,
Three cheers for a comment line. -- David Kastrup _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
