Looks good. I had one comment. Feel free to add an entry to the changelog. It's done by editing the file Documentation/changes.tely
Thanks, Carl http://codereview.appspot.com/4182056/diff/7001/scm/define-markup-commands.scm File scm/define-markup-commands.scm (right): http://codereview.appspot.com/4182056/diff/7001/scm/define-markup-commands.scm#newcode3403 scm/define-markup-commands.scm:3403: (define-markup-command (pattern layout props pattern count space) Now that I see it written -- should there be a direction argument for pattern? Or at least should we document that it's a horizontal repeat? http://codereview.appspot.com/4182056/diff/7001/scm/define-markup-commands.scm#newcode3416 scm/define-markup-commands.scm:3416: (if (zero? i) Nicely done! http://codereview.appspot.com/4182056/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel
