https://codereview.appspot.com/7185044/diff/45018/input/regression/tuplet-bracket-outside-staff-priority.ly
File input/regression/tuplet-bracket-outside-staff-priority.ly (right):

https://codereview.appspot.com/7185044/diff/45018/input/regression/tuplet-bracket-outside-staff-priority.ly#newcode17
input/regression/tuplet-bracket-outside-staff-priority.ly:17:
\tupletOutsideStaffPriority #1
Why a changed regtest when purportedly, according to the changes entry,
the functionality should not be changed?

https://codereview.appspot.com/7185044/diff/45018/input/regression/tuplet-number-outside-staff-positioning.ly
File input/regression/tuplet-number-outside-staff-positioning.ly
(right):

https://codereview.appspot.com/7185044/diff/45018/input/regression/tuplet-number-outside-staff-positioning.ly#newcode14
input/regression/tuplet-number-outside-staff-positioning.ly:14:
\tupletOutsideStaffPriority #1
Same here.

https://codereview.appspot.com/7185044/diff/45018/input/regression/tuplet-number-outside-staff-priority.ly
File input/regression/tuplet-number-outside-staff-priority.ly (right):

https://codereview.appspot.com/7185044/diff/45018/input/regression/tuplet-number-outside-staff-priority.ly#newcode12
input/regression/tuplet-number-outside-staff-priority.ly:12:
\tupletOutsideStaffPriority #1
same here.

https://codereview.appspot.com/7185044/diff/45018/ly/music-functions-init.ly
File ly/music-functions-init.ly (right):

https://codereview.appspot.com/7185044/diff/45018/ly/music-functions-init.ly#newcode1376
ly/music-functions-init.ly:1376: tupletOutsideStaffPriority =
Does that mean that the statement in changes.tely about tuplets'
reaction to outside-staff-priority being the same as before is false?

This kind of function does not make any sense.  For one thing, it does
not serve a high level function but manipulates a single low-level
parameter.  Providing a music function for this kind of numerical
manipulation is decidedly weird.  For another, it only provides an
override method while this parameter will often be in need of tweaking.

https://codereview.appspot.com/7185044/

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

Reply via email to