Re: Doc: NR 1: Use @address@hidden for variables. (issue2217046)

[percival . music . ca]

LGTM, although I didn't check that it still compiles.


http://codereview.appspot.com/2217046/diff/1/Documentation/notation/pitches.itely
File Documentation/notation/pitches.itely (right):

http://codereview.appspot.com/2217046/diff/1/Documentation/notation/pitches.itely#newcode142
Documentation/notation/pitches.itely:142:
Why the extra newline ?  I mean, it doesn't hurt anything, but all the
other @item's [sic] don't have any extra lines between the @item and the
text, and that's what the doc policy says to do.

http://codereview.appspot.com/2217046/diff/1/Documentation/notation/repeats.itely
File Documentation/notation/repeats.itely (right):

http://codereview.appspot.com/2217046/diff/1/Documentation/notation/repeats.itely#newcode682
Documentation/notation/repeats.itely:682: The same output can be
obtained by adding @samp{:@var{number}}
Why is this @samp instead of @code?
(I totally agree that the previous @q{} was wrong)

http://codereview.appspot.com/2217046/