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

fix: Some code that should have been destined for /v4 was instead put… #4154

Merged
merged 1 commit into from
Mar 5, 2023

Conversation

jimidle
Copy link
Collaborator

@jimidle jimidle commented Mar 5, 2023

fix: Code that should have gone to the /v4 files was instead placed in to the root (pre v4) files

This PR merely copies the changes in to the correct place - code has already been reviewed.

  • Incorporate an iterative tree walker
  • Expose Reset() and String() in lexer/token and add test

@parrt - please merge so I can then proceed with removing the pre-v4 files from the repo

… in to the root (pre v4) files

 - Incorporate an interative tree walker
 - Expose Reset() and String() in lexer/token and add test

Signed-off-by: Jim.Idle <jimi@idle.ws>
@parrt parrt added this to the 4.12.1 milestone Mar 5, 2023
@parrt parrt merged commit 40554bd into antlr:dev Mar 5, 2023
@parrt
Copy link
Member

parrt commented Mar 5, 2023

thanks!

@jimidle jimidle deleted the feature/v4catchup branch March 5, 2023 09:45
jimidle pushed a commit to jimidle/antlr4 that referenced this pull request Mar 28, 2023
… in to the root (pre v4) files (antlr#4154)

- Incorporate an interative tree walker
 - Expose Reset() and String() in lexer/token and add test

Signed-off-by: Jim.Idle <jimi@idle.ws>
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.

2 participants