-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Assign token names to string literals in parse rule contexts #2358
Assign token names to string literals in parse rule contexts #2358
Conversation
Hmm...well, this would have to be implemented in all other targets optimally. I am not sure diverging even more with Java target is worth the increased value. |
It works for all targets for now. But I'll test it on our big grammars set ASAP. |
Oh, i get it. it doesn't change code gen, it just makes it look like the token ref had a label. ok. please do test on a few big grammars. thanks. |
…F. Add getRuleTokens method
…rrect processing of arrays of string literals
… AssignTokenNamesToStringLiteralArraysInGeneratedParserRuleContexts unit-tests
Eventually, I've checked the new version on our grammars-v4 repository on Java runtime. Now I can say that everything is fine and I suggest to merge it :) |
8803407
to
2f7c5bf
Compare
fixes #2347