[Lilypond-auto] Issue 2854 in lilypond: Patch: parser.yy: remove STRING_IDENTIFIER token

[lilypond]

Status: New
Owner: ----
Labels: Type-Enhancement Patch-new

New issue 2854 by address@hidden: Patch: parser.yy: remove STRING_IDENTIFIER  
token
http://code.google.com/p/lilypond/issues/detail?id=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.

http://codereview.appspot.com/6542057