[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Documentation: elaborate NR `custom percussion’ section (issue 550650043
|
From: |
lemzwerg |
|
Subject: |
Documentation: elaborate NR `custom percussion’ section (issue 550650043 by address@hidden) |
|
Date: |
Fri, 26 Apr 2019 10:13:12 -0700 |
LGTM, thanks!
https://codereview.appspot.com/550650043/diff/548760043/Documentation/notation/percussion.itely
File Documentation/notation/percussion.itely (right):
https://codereview.appspot.com/550650043/diff/548760043/Documentation/notation/percussion.itely#newcode423
Documentation/notation/percussion.itely:423: @code{drumStyleTable}
property may then be set to use.
I think this sentence should end with `... be set to', dropping the word
`use'.
https://codereview.appspot.com/550650043/diff/548760043/Documentation/notation/percussion.itely#newcode463
Documentation/notation/percussion.itely:463: #(append
perhaps indenting by two spaces?
https://codereview.appspot.com/550650043/diff/548760043/Documentation/notation/percussion.itely#newcode515
Documentation/notation/percussion.itely:515: midiDrumPitches.dbass = g
since you align the `=' characters above, you should it do here, too.
https://codereview.appspot.com/550650043/
| [Prev in Thread] |
Current Thread |
[Next in Thread] |
- Documentation: elaborate NR `custom percussion’ section (issue 550650043 by address@hidden),
lemzwerg <=