Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)

2012-05-05 Thread graham
LGTM http://codereview.appspot.com/5976056/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)

2012-05-02 Thread tdanielsmusic
Hi James LGTM Thanks Trevor http://codereview.appspot.com/5976056/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)

2012-05-01 Thread pkx166h
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.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5976056/diff/1/Documentation/notation/

Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)

2012-04-29 Thread tdanielsmusic
LGTM, apart from the deleted index entries http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.itely#newcode1415 Documentation/notation/rh

Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)

2012-04-28 Thread pkx166h
Reviewers: Graham Percival, Trevor Daniels, Message: Thanks Trevor, this one slipped past my radar. http://codereview.appspot.com/5976056/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5976056/diff/1/Documentation/no

Re: Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)

2012-04-01 Thread tdanielsmusic
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.itely File Documentation/notation/rhythms.itely (right): http://codereview.appspot.com/5976056/diff/1/Documentatio

Doc: NR 1.2.3 Expl. manual accidentals + cadenzas (issue 5976056)

2012-04-01 Thread graham
LGTM, please push immediately assuming it passes make doc. http://codereview.appspot.com/5976056/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel