Re: Fix 1456 (issue3693042)

[n . puttock]

http://codereview.appspot.com/3693042/diff/1/Documentation/notation/rhythms.itely
File Documentation/notation/rhythms.itely (left):

http://codereview.appspot.com/3693042/diff/1/Documentation/notation/rhythms.itely#oldcode1059
Documentation/notation/rhythms.itely:1059: \relative c' {
\new Staff

http://codereview.appspot.com/3693042/diff/1/Documentation/notation/rhythms.itely#oldcode1106
Documentation/notation/rhythms.itely:1106: \relative c' {
doesn't need \new Staff, but might add for consistency with preceding
example

http://codereview.appspot.com/3693042/diff/1/input/regression/multiple-time-sig-settings.ly
File input/regression/multiple-time-sig-settings.ly (left):

http://codereview.appspot.com/3693042/diff/1/input/regression/multiple-time-sig-settings.ly#oldcode11
input/regression/multiple-time-sig-settings.ly:11: \relative c' {
needs \new Staff

http://codereview.appspot.com/3693042/diff/1/input/regression/skiptypesetting-multimeasurerest.ly
File input/regression/skiptypesetting-multimeasurerest.ly (right):

http://codereview.appspot.com/3693042/diff/1/input/regression/skiptypesetting-multimeasurerest.ly#newcode10
input/regression/skiptypesetting-multimeasurerest.ly:10: \version
"2.13.43"
On 2010/12/17 13:53:37, Graham Percival wrote:
> I guess this should be .44 ?  or .36 ?  I mean, nothing's changed in
this
> file...?

It should be removed.  There's already a version string at the top of
the file.

http://codereview.appspot.com/3693042/diff/1/ly/music-functions-init.ly
File ly/music-functions-init.ly (right):

http://codereview.appspot.com/3693042/diff/1/ly/music-functions-init.ly#newcode747
ly/music-functions-init.ly:747: (revert-time-signature-setting
time-signature))
indent

http://codereview.appspot.com/3693042/diff/1/scm/time-signature-settings.scm
File scm/time-signature-settings.scm (left):

http://codereview.appspot.com/3693042/diff/1/scm/time-signature-settings.scm#oldcode294
scm/time-signature-settings.scm:294: (if (and (pair? rest) (symbol? (car
rest)))
remove (+ `rest' arg)

http://codereview.appspot.com/3693042/diff/1/scm/time-signature-settings.scm#oldcode304
scm/time-signature-settings.scm:304: time-signature . rest)
remove `rest' arg

http://codereview.appspot.com/3693042/diff/1/scm/time-signature-settings.scm#oldcode312
scm/time-signature-settings.scm:312: (if (and (pair? rest) (symbol? (car
rest)))
remove

http://codereview.appspot.com/3693042/