lilypond-devel
[Top][All Lists]
Advanced

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

Die, instrumentSwitch. (issue 133390043 by address@hidden)


From: tdanielsmusic
Subject: Die, instrumentSwitch. (issue 133390043 by address@hidden)
Date: Sun, 31 Aug 2014 08:06:46 +0000

A few minor comments below, but I don't see
how we can just remove the instructions for
setting up InstrumentSwitch without some replacement.
InstrumentSwitch is used by \cueDuring and friends.

Trevor



https://codereview.appspot.com/133390043/diff/1/Documentation/notation/staff.itely
File Documentation/notation/staff.itely (right):

https://codereview.appspot.com/133390043/diff/1/Documentation/notation/staff.itely#newcode979
Documentation/notation/staff.itely:979: \transposition c'
I think a piccolo sounds an octave higher than written,
and probably an octave higher than the notes as entered,
so this should be c''.  Might be worth a note that the
converse needs to be specified when reverting to flute
or other non-transposing instrument.

https://codereview.appspot.com/133390043/diff/1/Documentation/notation/staff.itely#newcode992
Documentation/notation/staff.itely:992: \set Staff.shortInstrumentName =
#"Clt."
Shouldn't these two lines be removed?

https://codereview.appspot.com/133390043/diff/1/Documentation/notation/staff.itely#newcode1331
Documentation/notation/staff.itely:1331: s1*0^\markup { \tiny "flute" }
I prefer <>

https://codereview.appspot.com/133390043/diff/1/Documentation/notation/staff.itely#newcode1357
Documentation/notation/staff.itely:1357: s1*0^\markup { \tiny "flute" }
I prefer <>

https://codereview.appspot.com/133390043/diff/1/Documentation/notation/staff.itely#newcode1433
Documentation/notation/staff.itely:1433: s1*0^\markup { \tiny "flute" }
I prefer <>

https://codereview.appspot.com/133390043/



reply via email to

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