Doc: NR 1.2.3 Expl. manual accidentals + cadenzas

Doc: NR 1.2.3 Expl. manual accidentals + cadenzas

Issue 2438

Added notes about using manual accidentals for reminder accidentals
when using \cadenzaOn.

Add it to note about manual beams.

Tidied up some 'third person' references and tightened up sentences
along with some 'long' @cindex entries.

Added an @ref{}

[Graham Percival, 2012-04-01 18:02:48]

LGTM, please push immediately assuming it passes make doc.

[Trevor Daniels, 2012-04-01 22:09:16]

A couple of these changes are actually misleading, so I think they at least
should be corrected.

Trevor

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1415: @cindex beams, unmetered music
I think there should be index entries leading to this section starting from both
bar line and bar number, as before.  Why delete them?

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1422: In metered music, measures are
calculated automatically for bar lines
I don't know what "calculating measures" means.  Perhaps say, "In metered music
bar lines are inserted and bar numbers are calculated automatically."

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1450: Cadenzas are treated as a @q{single
measure} of music, even if one or
No, they can be shorter and within a measure of music.  Perhaps say, "A new bar
is never started within a cadenza, even if one or more @code{\bar} commands are
inserted within it."

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1478: These commands affect all staves in
the score, even when placed in just
We conventionally use the term "predefined commands" to refer to commands
defined in ly/property-init.ly.

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1512: In unmetered music, line and page
breaks will only occur at a bar line,
This is true generally, not just in unmetered music.
"Automatic line and page breaks are inserted only at bar lines, so @q{invisible}
bar lines will need to be inserted manually in long stretches of unmetered music
to permit breaking:"

[pkx166h, 2012-04-28 15:15:09]

Thanks Trevor, this one slipped past my radar.

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1415: @cindex beams, unmetered music
On 2012/04/01 22:09:16, Trevor Daniels wrote:
> I think there should be index entries leading to this section starting from
both
> bar line and bar number, as before.  Why delete them?
It was an oversight inasmuch as I meant to put back bar line as I have 'bar
numbers. Thanks for spotting.

I deleted the 'turning off' reference as that isn't what cadenza's do, they
force LP to 'ignore' them but not turn them off.

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1422: In metered music, measures are
calculated automatically for bar lines
On 2012/04/01 22:09:16, Trevor Daniels wrote:
> I don't know what "calculating measures" means.  Perhaps say, "In metered
music
> bar lines are inserted and bar numbers are calculated automatically."

Done.

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1450: Cadenzas are treated as a @q{single
measure} of music, even if one or
On 2012/04/01 22:09:16, Trevor Daniels wrote:
> No, they can be shorter and within a measure of music.  Perhaps say, "A new
bar
> is never started within a cadenza, even if one or more @code{\bar} commands
are
> inserted within it."

Done.

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1478: These commands affect all staves in
the score, even when placed in just
On 2012/04/01 22:09:16, Trevor Daniels wrote:
> We conventionally use the term "predefined commands" to refer to commands
> defined in ly/property-init.ly.

Of course! Thanks. Done.

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1512: In unmetered music, line and page
breaks will only occur at a bar line,
On 2012/04/01 22:09:16, Trevor Daniels wrote:
> This is true generally, not just in unmetered music.
> "Automatic line and page breaks are inserted only at bar lines, so
@q{invisible}
> bar lines will need to be inserted manually in long stretches of unmetered
music
> to permit breaking:"  

Done.

[Trevor Daniels, 2012-04-29 09:18:28]

LGTM, apart from the deleted index entries

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:1415: @cindex beams, unmetered music
The change you've made was not the one I wanted.  There are already several
adjacent entries in the index from 'cadenza' and 'unmetered', but now there are
none from 'bar lines' and 'bar numbers'.  There should be.  If you don't like
'turning off', how about simply 
@cindex bar lines, cadenzas
@cindex bar numbers, cadenzas
ditto for unmetered music
to go with the ones for beams and accidentals?

[pkx166h, 2012-05-02 06:17:37]

On 2012/04/29 09:18:28, Trevor Daniels wrote:
> LGTM, apart from the deleted index entries
> 
>
http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
> File Documentation/notation/rhythms.itely (right):
> 
>
http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.i...
> Documentation/notation/rhythms.itely:1415: @cindex beams, unmetered music
> The change you've made was not the one I wanted.  There are already several
> adjacent entries in the index from 'cadenza' and 'unmetered', but now there
are
> none from 'bar lines' and 'bar numbers'.  There should be.  If you don't like
> 'turning off', how about simply 
> @cindex bar lines, cadenzas
> @cindex bar numbers, cadenzas
> ditto for unmetered music
> to go with the ones for beams and accidentals?

Sounds good. Thanks.

[Trevor Daniels, 2012-05-02 08:55:33]

Hi James

LGTM

Thanks
Trevor

[Graham Percival, 2012-05-05 09:28:16]

LGTM

[pkx166h, 2012-05-09 06:03:05]

committer	James Lowe <pkx166h@gmail.com>	
Wed, 9 May 2012 06:00:15 +0000 (07:00 +0100)
commit	6cc24e4a3246f1daaf79db3bec97a4a4541b5149