Ken Domino

Results 839 comments of Ken Domino

I was thinking the same thing, that the recovery does not work well with ambiguous grammars. But, then, I checked parsing for `function onAction (event) { }` and there is...

I cannot reproduce the errors you give: * for the code in [core/src/main/resources/org/apache/spark/ui/static](https://github.com/apache/spark/tree/a881438114ea3e8e918d981ef89ed1ab956d6fca/core/src/main/resources/org/apache/spark/ui/static) * with grammar [javascript/javascript](https://github.com/antlr/grammars-v4/tree/18d8d8962f2876f555105042901b39d210c746e9/javascript/javascript) * for either target CSharp or Java. * using 4.13.1. * using trgen-generated...

As far as using the grammars, see https://stackoverflow.com/a/50438417/4779853. The author gives a link to code that uses both grammars.

Strictly speaking, if you are using [gfortran](https://gcc.gnu.org/wiki/GFortran#Using_gfortran) and not using the `-std` option, you're not forcing the compiler to choose a particular language compliance. NB: gfortran does not strictly support...

The parse for examples/deno_core_runtime.rs is `6 s` for CSharp, so it will be painfully slow for TypeScript/JavaScript. The main problem with the [rust/](https://github.com/antlr/grammars-v4/tree/c7e3f880a8fb247692d90ea6c4260149b75ebf2e/rust) grammar is that [statements](https://github.com/antlr/grammars-v4/blob/c7e3f880a8fb247692d90ea6c4260149b75ebf2e/rust/RustParser.g4#L495) and [expression](https://github.com/antlr/grammars-v4/blob/c7e3f880a8fb247692d90ea6c4260149b75ebf2e/rust/RustParser.g4#L395C5-L395C5) are...

The XQuery grammar doesn't note it, but it includes almost all the rules for the XPath grammar. See https://www.w3.org/TR/xpath-31/. Therefore, there's going to be a tension between writing this grammar...

The Spec divides the C++ grammar into two phases, with the grammar shared between both. But, the driver here is not set up for two parsers, where the output is...

> No newline at end of file Every single error message from the parser must end with a newline. You should use trgen to remaster your test files. I don't...

NB!! You placed an entirely new grammar for C in the same directory containing an existing grammar. They are not the same and it confuses everyone--and the build tool. The...

This looks good @borneq I tested out the grammar with all the targets, and it looks like Cpp, CSharp, Dart, Java all parse fast. The Go target works fast too,...