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/rhythms.itely#newcode1415
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. http://codereview.appspot.com/5976056/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
