https://codereview.appspot.com/557410043/diff/545570043/lily/parser.yy
File lily/parser.yy (right):

https://codereview.appspot.com/557410043/diff/545570043/lily/parser.yy#newcode3676
lily/parser.yy:3676: | pitch exclamations questions octave_check
duration stray_quotes optional_rest post_events {
why can't this case be folded in to the preceding rule? Maybe this
comment says it already; if so an example would help.

https://codereview.appspot.com/557410043/diff/545570043/lily/parser.yy#newcode3706
lily/parser.yy:3706: parser->parser_error (@6, _ ("octave marks must
precede duration"));
add a comment that we sholudn't drop this error, even though this works
for users (I think we don't want that because it makes parsing .ly
harder for others.)

https://codereview.appspot.com/557410043/

Reply via email to