Strumenta / antlr-kotlin

Support for Kotlin as a target for ANTLR 4
Apache License 2.0
221 stars 47 forks source link

Add a grammar test for TSQL #152

Closed lppedd closed 7 months ago

lppedd commented 7 months ago

This PR adds a test for the TSQL grammar, probably the largest grammar you can find in the ANTLR Grammars repository.

The generated file is so big it causes a o.j.o.objectweb.asm.MethodTooLargeException from the Kotlin compiler when targeting the JVM, so the testing is enabled only for the JS platform for now.

I will report MethodTooLargeException in YouTrack soon.

lppedd commented 7 months ago

As expected it is failing on the JVM. Let's keep this PR parked.

lppedd commented 7 months ago

@KvanTTT FYI

lppedd commented 7 months ago

@ftomassetti to recap the commits here:

I think we can merge as testing with a grammar this large will save us from many other issues in the future.

ftomassetti commented 7 months ago

Thank you @lppedd and @KvanTTT

I added minor comments