Skip to content

Conversation

dhbaird
Copy link

@dhbaird dhbaird commented Apr 21, 2017

Fix for exprs rule:

 exprs  
        : exprs COMMA expr
-               { $$ = $1; $$.push($1); }
+               { $$ = $1; $$.push($3); }
        | expr  
                { $$ = [$1]; }
        ;

To test out the fix:

$ gulp jison-compile
$ gulp js-merge
$ ( cd test/parser && mocha test003.js  )

  1 passing (15ms)

@dhbaird
Copy link
Author

dhbaird commented Apr 21, 2017

Fixes this query (the GROUP BY won't parse correctly without this patch):

SELECT a, b FROM T GROUP BY a, b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant