https://codereview.appspot.com/10473043/diff/1/Documentation/extending/scheme-tutorial.itely File Documentation/extending/scheme-tutorial.itely (right): https://codereview.appspot.com/10473043/diff/1/Documentation/extending/scheme-tutorial.itely#newcode1320 Documentation/extending/scheme-tutorial.itely:1320: @code{SequentialMusic} with the two @code{NoteEvent}. For adding to a ...
10 years, 10 months ago
(2013-06-24 09:23:16 UTC)
#2
https://codereview.appspot.com/10473043/diff/1/Documentation/extending/scheme...
File Documentation/extending/scheme-tutorial.itely (right):
https://codereview.appspot.com/10473043/diff/1/Documentation/extending/scheme...
Documentation/extending/scheme-tutorial.itely:1320: @code{SequentialMusic} with
the two @code{NoteEvent}. For adding to a
Awkward grammar which is probably the reason that the previous version chose the
grammatically nicer (and factually wrong) "@code{EventChords}". The
"programmer" fix would be "@code{NoteEvent}s" but I think I prefer
"@code{NoteEvent} elements". While it would be feasible to put "@code{...}"
around "elements" in this usage as it is properly the name of the
@code{SequentialMusic} property, I think that is too clever. So for me, just
"@code{NoteEvent} elements" would be best here.
At any rate: thanks for catching that oversight.
Issue 10473043: Changing EventChord to NoteEvent in scheme-tutorial.itely
(Closed)
Created 10 years, 10 months ago by thomasmorley651
Modified 10 years, 10 months ago
Reviewers: dak, Graham Percival
Base URL:
Comments: 1