Re: [Lilypond-auto] Issue 3401 in lilypond: Parser error with void function in music list

[lilypond]

Updates:
        Labels: Patch-new

Comment #1 on issue 3401 by address@hidden: Parser error with void function in  
music list
http://code.google.com/p/lilypond/issues/detail?id=3401#c1

Issue 3401: Parser error with void function in music list

Problem code fixed:

test =
   (symbol-list?))

{
  \test Symbol
}

This patch is sort of a so-so fix for the problem: for one thing,
embedded_scm_closed should not have been able to end in a symbol list
in the first place since parsing those requires lookahead for '.'.
The grammar has now been changed to use embedded_scm, and just backs
up in the case a scheme function/identifier returns a markup in lyrics
mode (which can then be assembled with duration and post-events into a
more complex music expression).  If parsing the source for the markup
required lookahead, the same error will be thrown.

But that use case of Scheme functions in Lyrics mode for generating
markups to be turned into lyric events is a lot more exotic than the
one triggering this issue.  The current fix at least results in
predictable behavior outside of using Lyrics mode, and in particular
for uses of define-void-function.

http://codereview.appspot.com/10081043

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings