You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucy.apache.org by "Nick Wellnhofer (JIRA)" <ji...@apache.org> on 2018/02/08 15:36:00 UTC
[lucy-issues] [jira] [Resolved] (LUCY-327) t/core/035-json.t aborts when compiled
with lemon 3.22.0
[ https://issues.apache.org/jira/browse/LUCY-327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Nick Wellnhofer resolved LUCY-327.
----------------------------------
Resolution: Fixed
Fix Version/s: 0.6.2
0.7.0
This was a bug in Lucy. Fixed here:
[https://git1-us-west.apache.org/repos/asf?p=lucy.git;a=commitdiff;h=e011b963e34890921f87942d0073b56a81e2c6cf]
And in the 0.6 branch:
[https://git1-us-west.apache.org/repos/asf?p=lucy.git;a=commitdiff;h=3a9128ce1f957140a14ad1aaf0bc7bdcfe41f9b5]
Thanks for the report!
> t/core/035-json.t aborts when compiled with lemon 3.22.0
> --------------------------------------------------------
>
> Key: LUCY-327
> URL: https://issues.apache.org/jira/browse/LUCY-327
> Project: Lucy
> Issue Type: Bug
> Components: Perl bindings
> Affects Versions: 0.6.1
> Reporter: Petr Pisar
> Priority: Minor
> Fix For: 0.7.0, 0.6.2
>
>
> When building Lucy-0.6.1 using lemon-3.22.0, t/core/035-json.t aborts:
> {noformat}
> t/core/033-index_file_names.t ......... ok
> perl: cfcore/Lucy/Util/Json/JsonParser.c:681: yy_find_shift_action: Assertion `iLookAhead < YYNTOKEN' failed.
> t/core/035-json.t .....................
> Failed 43/135 subtests
> {noformat}
> This does not happen with lemon-3.20.1.
> The generated yy_find_shift_action() function has changed like this:
> {noformat}
> @@ -617,13 +669,18 @@ static unsigned int yy_find_shift_action
> int i;
> int stateno = pParser->yytos->stateno;
>
> - if( stateno>=YY_MIN_REDUCE ) return stateno;
> + if( stateno>YY_MAX_SHIFT ) return stateno;
> assert( stateno <= YY_SHIFT_COUNT );
> +#if defined(YYCOVERAGE)
> + yycoverage[stateno][iLookAhead] = 1;
> +#endif
> do{
> i = yy_shift_ofst[stateno];
> + assert( i>=0 && i+YYNTOKEN<=sizeof(yy_lookahead)/sizeof(yy_lookahead[0]) );
> assert( iLookAhead!=YYNOCODE );
> + assert( iLookAhead < YYNTOKEN );
> i += iLookAhead;
> - if( i<0 || i>=YY_ACTTAB_COUNT || yy_lookahead[i]!=iLookAhead ){
> + if( yy_lookahead[i]!=iLookAhead ){
> #ifdef YYFALLBACK
> YYCODETYPE iFallback; /* Fallback token */
> if( iLookAhead<sizeof(yyFallback)/sizeof(yyFallback[0])
> {noformat}
> I don't know if this is a bug in the lemon or Lucy. sqlite-3.22.0 changelog only mentions lemon performance improvements. I believe this is in issue in the lemon, but I think you should know about that.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)