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-12 Thread Richard Hipp
On 7/9/16, Nick Wellnhofer wrote: > > This still doesn't work for me. OK. Another fix. Please try the latest trunk version. Note to passive readers of this thread: none of this has any impact on SQLite. SQLite does not use the feature of the Lemon LALR(1) parser generator that Nick is fixing.

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. Thanks

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

2016-07-08 Thread Richard Hipp
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. Thanks. > > Still doesn't work for me. The

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 Richard Hipp
Please try https://www.sqlite.org/src/info/2683b375ad129117 and verify that the changes on trunk are working. Thanks. On 7/5/16, Nick Wellnhofer wrote: > 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

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" branch as well, everything works as expecte

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

2016-07-05 Thread Richard Hipp
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" branch as well, everything works as > expected. Can you show me your patch, plea

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

2016-07-05 Thread Nick Wellnhofer
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" branch as well, everything works as expected. Thanks for the quick response! Nick On 05/07/2016 14:48, Richard Hipp

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

2016-07-05 Thread Richard Hipp
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 > upgrading to the latest version

[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=