LGTM

http://codereview.appspot.com/4810072/diff/3002/input/regression/slur-height-capping.ly
File input/regression/slur-height-capping.ly (right):

http://codereview.appspot.com/4810072/diff/3002/input/regression/slur-height-capping.ly#newcode4
input/regression/slur-height-capping.ly:4: texidoc = "Lilypond ignores
extra-collision-objects towards
"slur shape is not adapted to accomodate objects towards the edges of
the slurs"

(we should still score for collisions)

http://codereview.appspot.com/4810072/diff/3002/lily/slur-configuration.cc
File lily/slur-configuration.cc (right):

http://codereview.appspot.com/4810072/diff/3002/lily/slur-configuration.cc#newcode102
lily/slur-configuration.cc:102: close_to_edge = close_to_edge || -d *
(p[X_AXIS] - curve_xext[d]) < close_to_edge_length;
i'd simply use fabs.

http://codereview.appspot.com/4810072/

_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to