Github user rogersjeffreyl commented on a diff in the pull request:
https://github.com/apache/incubator-quickstep/pull/38#discussion_r67624252
--- Diff: parser/preprocessed/SqlLexer_gen.cpp ---
@@ -1300,11 +1280,16 @@ extern int quickstep_yylex \
*/
YY_DECL
{
- yy_state_type yy_current_state;
- char *yy_cp, *yy_bp;
- int yy_act;
+ register yy_state_type yy_current_state;
--- End diff --
@hbdeshmukh
The register variables are generated in my my machine as well ans I get a
compilation error. However it seems to have worked for you. Not sure if this is
due to the version of flex being used.
We can generate the modified preprocessed files on my machine and then
commit them
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---