Ken Domino

Results 839 comments of Ken Domino

You mean Bash diff? https://chatgpt.com/c/6935aca5-0950-8328-9f5d-f8a67c05e194

I don't have a good tool that computes actual usages from a collection of input test files. I punted on the computation in [trcover](https://github.com/kaby76/Trash/tree/main/src/trcover). That tool used to be called...

In the input string `extern int x;`, the `x` is parsed as a [declarationSpecifier](https://github.com/antlr/grammars-v4/blob/c164eb4228e925754c8b4ebab303d4be1cf3cf04/c/C.g4#L182). It should be parsed as an [initDeclaratorList](https://github.com/antlr/grammars-v4/blob/c164eb4228e925754c8b4ebab303d4be1cf3cf04/c/C.g4#L190). You can see this because the grammar is ambiguous:...

Yes, the grammar *cannot* work without semantics. The ISO Spec EBNF is insufficient to get the right parse. It is also the reason why the generated parse is slow, because...

You could try fiddling with the grammar and see if you can solve the problem. But, this is what I was thinking of. Add a predicate to the expression `declarationSpecifier+`...

I started PR https://github.com/antlr/grammars-v4/pull/4682 to fix this and other problems. I'm not sure whether I will fix the max-k problems in this round but at least some of the ambiguity....

I would think that the [version stated in the grammar](https://github.com/antlr/grammars-v4/blob/c164eb4228e925754c8b4ebab303d4be1cf3cf04/sql/plsql/PlSqlParser.g4#L2) is incorrect at this point. (It needs to be removed ASAP.) That said, the grammar has undergone incremental changes over...