Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(6731)

Issue 61530043: Doc: NR Clarify repeats w\ partials and barchecks (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 2 months ago by pkx166h
Modified:
10 years, 2 months ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Doc: NR Clarify repeats w\ partials and barchecks Repeats.itely Suggestions from Keith via another unrelated patch (but in the same *.itely file) to tidy up the two examples when not to use bar checks and partials with \alternative repeats.

Patch Set 1 #

Total comments: 4

Patch Set 2 : With Graham's corrections. #

Total comments: 6

Patch Set 3 : WIth Trevor's clarifying suggestions. #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -20 lines) Patch
M Documentation/notation/repeats.itely View 1 2 1 chunk +14 lines, -20 lines 2 comments Download

Messages

Total messages: 8
Graham Percival
https://codereview.appspot.com/61530043/diff/1/Documentation/notation/repeats.itely File Documentation/notation/repeats.itely (right): https://codereview.appspot.com/61530043/diff/1/Documentation/notation/repeats.itely#newcode167 Documentation/notation/repeats.itely:167: If a repeat, that has no alternate endings, starts ...
10 years, 2 months ago (2014-02-14 08:23:01 UTC) #1
pkx166h
https://codereview.appspot.com/61530043/diff/1/Documentation/notation/repeats.itely File Documentation/notation/repeats.itely (right): https://codereview.appspot.com/61530043/diff/1/Documentation/notation/repeats.itely#newcode167 Documentation/notation/repeats.itely:167: If a repeat, that has no alternate endings, starts ...
10 years, 2 months ago (2014-02-16 06:38:46 UTC) #2
pkx166h
With Graham's corrections.
10 years, 2 months ago (2014-02-16 06:39:44 UTC) #3
Trevor Daniels
A few suggestions for extra clarity ... https://codereview.appspot.com/61530043/diff/20001/Documentation/notation/repeats.itely File Documentation/notation/repeats.itely (right): https://codereview.appspot.com/61530043/diff/20001/Documentation/notation/repeats.itely#newcode168 Documentation/notation/repeats.itely:168: measure, usually ...
10 years, 2 months ago (2014-02-16 09:03:19 UTC) #4
pkx166h
WIth Trevor's clarifying suggestions.
10 years, 2 months ago (2014-02-16 16:30:24 UTC) #5
pkx166h
https://codereview.appspot.com/61530043/diff/20001/Documentation/notation/repeats.itely File Documentation/notation/repeats.itely (right): https://codereview.appspot.com/61530043/diff/20001/Documentation/notation/repeats.itely#newcode168 Documentation/notation/repeats.itely:168: measure, usually it will also end somewhere in the ...
10 years, 2 months ago (2014-02-16 16:30:45 UTC) #6
Graham Percival
https://codereview.appspot.com/61530043/diff/40001/Documentation/notation/repeats.itely File Documentation/notation/repeats.itely (right): https://codereview.appspot.com/61530043/diff/40001/Documentation/notation/repeats.itely#newcode186 Documentation/notation/repeats.itely:186: measure, measure, the same principles apply, except that a ...
10 years, 2 months ago (2014-02-17 13:40:40 UTC) #7
dak
10 years, 2 months ago (2014-02-17 13:57:36 UTC) #8
https://codereview.appspot.com/61530043/diff/40001/Documentation/notation/rep...
File Documentation/notation/repeats.itely (right):

https://codereview.appspot.com/61530043/diff/40001/Documentation/notation/rep...
Documentation/notation/repeats.itely:186: measure, measure, the same principles
apply, except that a
On 2014/02/17 13:40:40, Graham Percival wrote:
> unless my old eyes are misleading me, you have "measure," twice.
> 
> I guess you're doing some carpentry at home?  Measure twice, cut once?

When Gounod appropriated Bach's prelude #1 für Das Wohltemperirte Clavier for
his Ave Maria, he slid in an additional measure as well.  That seems like a more
significant precursor than carpentry.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b