|
From: | tdanielsmusic |
Subject: | LSR: Updated snippet for MMR Positions (1931) (issue 5155045) |
Date: | Sat, 01 Oct 2011 21:19:49 +0000 |
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/
[Prev in Thread] | Current Thread | [Next in Thread] |