NULL as a keyword in the target language breaks compilation

[jacobobq]

So... C++, right? Has a macro somewhere called NULL. And when tokens are generated in the source code of the parser they use the token itself as name. Now, SQL has a NULL keyword so trying to generate a grammar for any flavor of it fails because NULL is already defined.

For instance, take the Trino's (formerly PrestoSQL) grammar definition. This will fail to compile the generated .h/.cpp files.

Expected behaviour:
It compiles normally.

[jacobobq]

I'm trying to find a way to solve this (probably prefix all tokens) and I will try to send you a PR.

Currently no luck.

[camertron]

Hey @jacobobq, thanks for bringing this up. Symbol clashes like this are a known class of problem with ANTLR that is unlikely to be addressed. Here's a very similar issue to the one you brought up. As suggested, you might try adding an underscore to the end, eg. NULL_.