Skip to content
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

Export Go lexer methods #1996

Merged
merged 4 commits into from
Oct 21, 2017
Merged

Export Go lexer methods #1996

merged 4 commits into from
Oct 21, 2017

Conversation

ereyes01
Copy link
Contributor

I've exported several methods in the Go runtime. I found that these methods were used in some grammar actions in grammars I was porting to Go. The analogues of these methods in the Java runtime also appear to be public methods, so this change brings the Go runtime closer to parity with the Java runtime.

- They can be called in target-specific grammar actions, therefore must
  be exported.
- Their Java analogues are public methods.
@parrt parrt requested a review from pboyer October 10, 2017 19:52
@pboyer
Copy link
Contributor

pboyer commented Oct 11, 2017

Thanks @ereyes01! LGTM.

@parrt parrt added this to the 4.7.1 milestone Oct 21, 2017
@parrt parrt merged commit 5fe98b1 into antlr:master Oct 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants