LGTM, apart from a couple of nitpicks which I missed earlier. No need to post a new patch-set.
https://codereview.appspot.com/6948070/diff/6001/Documentation/notation/notation-appendices.itely File Documentation/notation/notation-appendices.itely (left): https://codereview.appspot.com/6948070/diff/6001/Documentation/notation/notation-appendices.itely#oldcode1545 Documentation/notation/notation-appendices.itely:1545: (BNF) in @ref{LilyPond grammar}. This file is used to build the Our standard is two spaces after full stops in mono-spaced text. https://codereview.appspot.com/6948070/diff/6001/Documentation/notation/notation-appendices.itely File Documentation/notation/notation-appendices.itely (right): https://codereview.appspot.com/6948070/diff/6001/Documentation/notation/notation-appendices.itely#newcode1534 Documentation/notation/notation-appendices.itely:1534: @cindex grammar, for LilyPond I think the comma is wrong here. "grammar for Lilypond" is perfectly sensible. https://codereview.appspot.com/6948070/diff/6001/Documentation/notation/notation-appendices.itely#newcode1547 Documentation/notation/notation-appendices.itely:1547: parser during the program build by the parser generator, Bison. It is Same here. https://codereview.appspot.com/6948070/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
