https://codereview.appspot.com/265410043/diff/60001/lily/part-combine-iterator.cc File lily/part-combine-iterator.cc (right):
https://codereview.appspot.com/265410043/diff/60001/lily/part-combine-iterator.cc#newcode146 lily/part-combine-iterator.cc:146: // get_event_length(), which reads "length"? On 2015/10/08 12:07:42, Trevor Daniels wrote:
Perhaps this question should be resolved before pushing.
That'll teach me to raise questions about the existing implementation. If someone can help answer this and explain that a specific and simple change is desirable, I'd be happy to take action. Otherwise, I'd be willing to retract the comment before pushing. Otherwise, this patch is going to be delayed until I have more time. https://codereview.appspot.com/265410043/ _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
