A few nitpicks.  BTW, expect to produce 3-6 drafts.

http://codereview.appspot.com/4005046/diff/1/ly/merge-function.ly
File ly/merge-function.ly (right):

http://codereview.appspot.com/4005046/diff/1/ly/merge-function.ly#newcode3
ly/merge-function.ly:3:
Having an example is a nice idea, but don't do it here.  Make a separate
file Documentation/snippets/new/merging-rests.ly or something like that.

http://codereview.appspot.com/4005046/diff/1/ly/merge-function.ly#newcode8
ly/merge-function.ly:8: \include "merge-rests"
The whole point of this work is so that you _won't_ need to \include
merge-rests, so I'd omit this part.  :)

http://codereview.appspot.com/4005046/

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

Reply via email to