Hi James, Thanks for doing this. It LGTM (but don't forget to remove the @knownissue)
Just two suggestions for you: a changes.itely entry is probably a good idea, and the @seealso for the Internals Reference needs the following items added: DoublePercentEvent DoubleRepeatSlash Double_percent_repeat_engraver RepeatSlashEvent Slash_repeat_engraver Cheers, Neil http://codereview.appspot.com/4273125/diff/1/Documentation/notation/repeats.itely File Documentation/notation/repeats.itely (right): http://codereview.appspot.com/4273125/diff/1/Documentation/notation/repeats.itely#newcode583 Documentation/notation/repeats.itely:583: \repeat percent 5 { c32 d e f} f } http://codereview.appspot.com/4273125/diff/1/Documentation/notation/repeats.itely#newcode584 Documentation/notation/repeats.itely:584: \repeat percent 4 { c16 d e f} f } http://codereview.appspot.com/4273125/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel
