lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

python/convertrules.ly: Use bare rhythms after ties for simple cases (is


From: tdanielsmusic
Subject: python/convertrules.ly: Use bare rhythms after ties for simple cases (issue 49470049)
Date: Fri, 10 Jan 2014 17:35:10 +0000


https://codereview.appspot.com/49470049/diff/1/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/49470049/diff/1/Documentation/learning/fundamental.itely#newcode1054
Documentation/learning/fundamental.itely:1054: r8 e4 d c8 ~ | 8 b16 a b8
g ~ 2 |
Didn't we decide a while ago to attach ties
to notes without an intervening space?  There
are obviously lots more like this too.

https://codereview.appspot.com/49470049/diff/1/Documentation/learning/templates.itely
File Documentation/learning/templates.itely (right):

https://codereview.appspot.com/49470049/diff/1/Documentation/learning/templates.itely#newcode62
Documentation/learning/templates.itely:62: BassMusic = \relative c' {
c2\p c4 c }
This is wrong.  Why should we not use the
new facility you introduced in 2.19 and
let the first note specify the octave?

https://codereview.appspot.com/49470049/diff/1/Documentation/learning/templates.itely#newcode82
Documentation/learning/templates.itely:82: BassMusic = \relative c' {
c2\p c4 c }
See above

https://codereview.appspot.com/49470049/diff/1/Documentation/learning/templates.itely#newcode126
Documentation/learning/templates.itely:126: SopranoMusic = \relative c'
{ a' a a a }
See above

https://codereview.appspot.com/49470049/diff/1/Documentation/learning/templates.itely#newcode127
Documentation/learning/templates.itely:127: AltoMusic = \relative c' {
f' f f f }
See above.
I'll not bother identifying any more

https://codereview.appspot.com/49470049/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]