LGTM; just a few nitpicks

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely
File Documentation/notation/spacing.itely (right):

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1297
Documentation/notation/spacing.itely:1297: c4 c c~ | \break  % this
\break works
indent; needs another c

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1298
Documentation/notation/spacing.itely:1298: c4 c  c c  |
spacing

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1316
Documentation/notation/spacing.itely:1316: c4 c  c c  |
do we need these two lines here?

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode1556
Documentation/notation/spacing.itely:1556: music = { c'8 c' c' c' }
\relative

http://codereview.appspot.com/3406041/diff/6001/Documentation/notation/spacing.itely#newcode2227
Documentation/notation/spacing.itely:2227: primarily concern the amount
of vertical padding different staves
primarily affect the amount of vertical padding between the staves and
sysems.

http://codereview.appspot.com/3406041/

_______________________________________________
lilypond-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to