[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/
- Die, instrumentSwitch. (issue 133390043 by address@hidden),
tdanielsmusic <=