Updates:
Status: Fixed
Labels: -Patch-push Fixed_2_17_4
Comment #4 on issue 2854 by [email protected]: Patch: parser.yy: remove
STRING_IDENTIFIER token
http://code.google.com/p/lilypond/issues/detail?id=2854
Pushed to staging as
commit aaa81de1e2e7e5c69b9ad3639f12227fb83c3828
Author: David Kastrup <[email protected]>
Date: Sat Sep 22 15:35:42 2012 +0200
Issue 2854: parser.yy: remove STRING_IDENTIFIER token
A STRING_IDENTIFIER always was a special case of a MARKUP_IDENTIFIER
without being treated equally at top level. That led to craziness
like
xxx=\markup "test"
\xxx
being invalid, while
xxx=\markup \italic "test"
\xxx
was valid code. This change just removes the STRING_IDENTIFIER
category altogether. Where it was used previously, MARKUP_IDENTIFIER
is accepted now syntactically, and where the semantics indeed require
a plain string, an error is generated explicitly in the action instead
of leaving it to the parser to complain about bad grammar.
