[sqlite] -Wsign-compare warning in lempar.c

2018-02-08 Thread Nick Wellnhofer
With the latest Lemon code, I get a warning under GCC with -Wsign-compare: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] assert( i>=0 && i+YYNTOKEN<=sizeof(yy_lookahead)/sizeof(yy_lookahead[0]) ); ^ Nick ___

Re: [sqlite] Lemon doesn't reset `yyerrcnt` after error

2016-07-14 Thread Nick Wellnhofer
On 12/07/2016 22:01, Richard Hipp wrote: OK. Another fix. Please try the latest trunk version. This version works for me. Thanks. Nick ___ sqlite-users mailing list sqlite-users@mailinglists.sqlite.org http://mailinglists.sqlite.org/cgi-bin/mailma

Re: [sqlite] Lemon doesn't reset `yyerrcnt` after error

2016-07-09 Thread Nick Wellnhofer
On 08/07/2016 21:54, Richard Hipp wrote: Please try again with the latest version of Lemon. Thanks. This still doesn't work for me. I created a GitHub repo to demonstrate the problem: https://github.com/nwellnhof/lemon-bug Nick ___ sqlite-us

Re: [sqlite] Lemon doesn't reset `yyerrcnt` after error

2016-07-09 Thread Nick Wellnhofer
On 08/07/2016 21:54, Richard Hipp wrote: Please try again with the latest version of Lemon. Thanks. On 7/6/16, Nick Wellnhofer wrote: On 05/07/2016 18:12, Richard Hipp wrote: Please try https://www.sqlite.org/src/info/2683b375ad129117 and verify that the changes on trunk are working

Re: [sqlite] Lemon doesn't reset `yyerrcnt` after error

2016-07-06 Thread Nick Wellnhofer
On 05/07/2016 18:12, Richard Hipp wrote: Please try https://www.sqlite.org/src/info/2683b375ad129117 and verify that the changes on trunk are working. Thanks. Still doesn't work for me. The structure of the #ifdefs in `Parse` is: #ifdef YYERRORSYMBOL ... #elif defined(YYNOERRORR

Re: [sqlite] Lemon doesn't reset `yyerrcnt` after error

2016-07-05 Thread Nick Wellnhofer
On 05/07/2016 17:15, Richard Hipp wrote: On 7/5/16, Nick Wellnhofer wrote: No, this doesn't fix my problem. The check-in only changes the "#ifdef YYERRORSYMBOL" branch which I don't define. But if I add the change to the "YYERRORSYMBOL is not defined" branc

Re: [sqlite] Lemon doesn't reset `yyerrcnt` after error

2016-07-05 Thread Nick Wellnhofer
05/07/2016 14:48, Richard Hipp wrote: Please try the latest check-in (https://www.sqlite.org/src/info/91889fa30e84760e) and let me know whether or not it clears your problem. On 7/5/16, Nick Wellnhofer wrote: Hello, I hope this is right place to report Lemon issues. I ran into a problem after u

[sqlite] Lemon doesn't reset `yyerrcnt` after error

2016-07-05 Thread Nick Wellnhofer
Hello, I hope this is right place to report Lemon issues. I ran into a problem after upgrading to the latest version of the Lemon source code from `trunk`. The following commit removed the initialization of `yyerrcnt` from the `Parse` function: http://www.sqlite.org/src/fdiff?sbs=1&v1=