A couple of nitpicks, otherwise LGTM (but untested).

(I'll abandon my patch - yours is better :)

Trevor


http://codereview.appspot.com/5155045/diff/1/Documentation/snippets/new/positioning-multi-measure-rests.ly
File Documentation/snippets/new/positioning-multi-measure-rests.ly
(right):

http://codereview.appspot.com/5155045/diff/1/Documentation/snippets/new/positioning-multi-measure-rests.ly#newcode1
Documentation/snippets/new/positioning-multi-measure-rests.ly:1:
\version "2.14.0"
2.15.14

http://codereview.appspot.com/5155045/diff/1/Documentation/snippets/new/positioning-multi-measure-rests.ly#newcode57
Documentation/snippets/new/positioning-multi-measure-rests.ly:57: %
Multi-Measure Rest by default are set under the fourth line
% Multi-measure rests ...

http://codereview.appspot.com/5155045/diff/1/Documentation/snippets/new/positioning-multi-measure-rests.ly#newcode77
Documentation/snippets/new/positioning-multi-measure-rests.ly:77: % Both
voices with a Multi Measure Rest
% Multi-measure rests in both voices remain separate

http://codereview.appspot.com/5155045/diff/1/Documentation/snippets/new/positioning-multi-measure-rests.ly#newcode80
Documentation/snippets/new/positioning-multi-measure-rests.ly:80: % More
that two voices with a Multi Measure Rest require an override
% Separating multi-measure rests in more than two voices requires an
override

http://codereview.appspot.com/5155045/

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

Reply via email to