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/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to