On 2015/09/02 10:13:26, benko.pal wrote:

https://codereview.appspot.com/264950043/diff/40001/lily/include/grob-array.hh#newcode63
lily/include/grob-array.hh:63: // Note: This method may reorder the
array
without affecting the result of
what does "reorder" mean in the current context?  the function
replaces some
grobs, deletes others.

If the sorting criterion uses attributes of the Grob that are changed by
replacement, the array might not be sorted afterward (or might be sorted
when it wasn't before).  Grob_array does not appear to dictate the
sorting criterion.

https://codereview.appspot.com/264950043/

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

Reply via email to