-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[antlr] add LexerAdaptor for dart #3977
base: master
Are you sure you want to change the base?
Conversation
I have a problem compiling the code. Also, the desc.xml should have "Dart" listed. All the driver code is in the repo, here: https://github.com/antlr/grammars-v4/tree/master/_scripts/templates/Dart |
Although the SDK version of the environment is 3.3.0-273.0. dev, this line should play a crucial role. |
I changed the pubspec.yaml as you suggested, but the parser is not producing the correct output. I am expecting errors for examples/LexerElementLabel.g4. The Antlr4 tool, and the CSharp and Java target parsers all report the error, but the Dart target does not. Please update your code and test.
|
Ok, I will revise it as soon as possible. Thanks for your feedback~ |
This PR may have some inappropriate aspects as I cannot find the contribution guide.
Please feel free to point it out.