Re: LSR: Updated snippet for MMR Positions (1931) (issue 5155045)

[pkx166h]

Reviewers: Trevor Daniels, Graham Percival,

Message:
Nitpicks done. I'm still not sure of the process here

Do I need to delete the file in ../snippets/.. and then re-rerun
makelsr.py, or what?

Also I guess we need to get phil to remove this deprecated snippet from
the LSR itself so it doesn't clobber this in git or doesn't it work like
that?

Thanks

James


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#newcode57
Documentation/snippets/new/positioning-multi-measure-rests.ly:57: %
Multi-Measure Rest by default are set under the fourth line
On 2011/10/01 21:19:50, Trevor Daniels wrote:
> % Multi-measure rests ...

Done.

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
On 2011/10/01 21:19:50, Trevor Daniels wrote:
> % Multi-measure rests in both voices remain separate

Done.

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
On 2011/10/01 21:19:50, Trevor Daniels wrote:
> % Separating multi-measure rests in more than two voices requires an
override

Done.

Description:
LSR: Updated snippet for MMR Positions (1931)

Removed statement that default MMR position was #0 (it is actually #2)

Added some examples of the overrides and clarified some comments
including adding new example of more than two voices

Please review this at http://codereview.appspot.com/5155045/

Affected files:
  A Documentation/snippets/new/positioning-multi-measure-rests.ly


Index: Documentation/snippets/new/positioning-multi-measure-rests.ly
diff --git a/Documentation/snippets/new/positioning-multi-measure-rests.ly  
b/Documentation/snippets/new/positioning-multi-measure-rests.ly
new file mode 100644
index  
0000000000000000000000000000000000000000..ca752462a02400f098aa1f755ac33f624fa05f61
--- /dev/null
+++ b/Documentation/snippets/new/positioning-multi-measure-rests.ly
@@ -0,0 +1,59 @@
+\version "2.15.14"
+
+\header {
+  lsrtags = "rhythms, tweaks-and-overrides"
+
+  texidoc = "
+Unlike ordinary rests, there is no predefined command to change the
+staff position of a multi-measure rest symbol of either form by
+attaching it to a note.  However, in polyphonic music multi-measure
+rests in odd-numbered and even-numbered voices are vertically
+separated. The positioning of multi-measure rests can be controlled as
+follows:
+
+"
+  doctitle = "Positioning multi-measure rests"
+} % begin verbatim
+
+\relative c'' {
+  % Multi-measure rests by default are set under the fourth line
+  R1
+  % They can be moved using an override
+  \override MultiMeasureRest #'staff-position = #-2
+  R1
+  \override MultiMeasureRest #'staff-position = #0
+  R1
+  \override MultiMeasureRest #'staff-position = #2
+  R1
+  \override MultiMeasureRest #'staff-position = #3
+  R1
+  \override MultiMeasureRest #'staff-position = #6
+  R1
+  \revert MultiMeasureRest #'staff-position
+  \break
+
+  % In two Voices, odd-numbered voices are under the top line
+  << { R1 } \\ { a1 } >>
+  % Even-numbered voices are under the bottom line
+  << { a1 } \\ { R1 } >>
+  % Multi-measure rests in both voices remain separate
+  << { R1 } \\ { R1 } >>
+
+  % Separating multi-measure rests in more than two voices
+  % requires an override
+  << { R1 } \\ { R1 } \\
+     \once \override MultiMeasureRest #'staff-position = #0
+     { R1 }
+  >>
+
+  % Using compressed bars in multiple voices requires another override
+  % in all voices to avoid multiple instances being printed
+  \compressFullBarRests
+  <<
+   \revert MultiMeasureRest #'staff-position
+    { R1*3 }
+    \\
+   \revert MultiMeasureRest #'staff-position
+    { R1*3 }
+  >>
+}