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

Augment error message during testing to include full cause of problem. #3843

Merged
merged 1 commit into from
Aug 27, 2022

Conversation

parrt
Copy link
Member

@parrt parrt commented Aug 27, 2022

Cause of compile/link errors were hidden during tests; bad in C++ as can't see anything.

@parrt
Copy link
Member Author

parrt commented Aug 27, 2022

See any issues @KvanTTT ?

@parrt parrt merged commit 34ca29a into antlr:dev Aug 27, 2022
@KvanTTT
Copy link
Member

KvanTTT commented Sep 5, 2022

Sorry, didn't notice the message at the time. Interesting, I thought toString returns exhaustive message.

@parrt
Copy link
Member Author

parrt commented Sep 5, 2022

no prob.

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