Regression in parser

2006-09-23 Thread Nicolas Sceaux
Hi, Using LilyPond from CVS, the following snippet causes a syntax error, which 2.9.18 did not: /** testFunction = #(define-music-function (parser location) () ;; test-data.ly contains music variable definitions (ly:parser-parse-string parser "\\include \"test-dat

Re: Regression in parser

2006-09-25 Thread Erik Sandberg
On Saturday 23 September 2006 17:49, Nicolas Sceaux wrote: > Hi, > > Using LilyPond from CVS, the following snippet causes a syntax error, > which 2.9.18 did not: I think the problem is my fix for music function arity: The extra tokens inserted by the lexer may disturb the processing of \include.

Re: Regression in parser

2006-09-25 Thread Nicolas Sceaux
Erik Sandberg <[EMAIL PROTECTED]> writes: > On Saturday 23 September 2006 17:49, Nicolas Sceaux wrote: >> Hi, >> >> Using LilyPond from CVS, the following snippet causes a syntax error, >> which 2.9.18 did not: > > I think the problem is my fix for music function arity: The extra tokens > inserte

Re: Regression in parser

2006-09-26 Thread Erik Sandberg
On Saturday 23 September 2006 17:49, Nicolas Sceaux wrote: > Hi, > > Using LilyPond from CVS, the following snippet causes a syntax error, > which 2.9.18 did not: > > /** > testFunction = > #(define-music-function (parser location) () > ;; test-data.ly contains music v

Re: Regression in parser

2006-09-26 Thread Han-Wen Nienhuys
Erik Sandberg schreef: On Saturday 23 September 2006 17:49, Nicolas Sceaux wrote: Hi, Using LilyPond from CVS, the following snippet causes a syntax error, which 2.9.18 did not: /** testFunction = #(define-music-function (parser location) () ;; test-data.ly contai

Re: Regression in parser

2006-09-27 Thread Erik Sandberg
On Tuesday 26 September 2006 23:54, Han-Wen Nienhuys wrote: > Erik Sandberg schreef: > > Hi, > > > > You should use ly:clone-parser to avoid this problem: > > > > #(define-music-function (parser location) () > >(ly:parser-parse-string (ly:clone-parser parser) "\\include > > \"test-data.ly\"") >

Re: Regression in parser

2006-09-27 Thread Han-Wen Nienhuys
Erik Sandberg schreef: The new thing now is that also 0-ary functions are subject to the same problem: We need to read an extra token to see whether an EXPECT_XXX comes. If you like, I can fix this particular case by adding a special token for 0-ary functions (but this will not fix the underlyi

Re: Regression in parser

2006-09-30 Thread Nicolas Sceaux
Erik Sandberg <[EMAIL PROTECTED]> writes: > On Saturday 23 September 2006 17:49, Nicolas Sceaux wrote: >> Hi, >> >> Using LilyPond from CVS, the following snippet causes a syntax error, >> which 2.9.18 did not: >> >> /** >> testFunction = >> #(define-music-function (par