CleanroomMC / GroovyScript

A scripting mod for Minecraft without the issues of other scripting mods.
GNU Lesser General Public License v3.0
40 stars 28 forks source link

Incorrect error from Error Catcher #270

Closed ghzdude closed 1 week ago

ghzdude commented 1 week ago

if you have an event/closure like:

mods.gregtech.materialEvent {
  method(, arg1, arg2)
}

the error incorrectly points to { as the issue, instead of the errant comma in method()

org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
  Unexpected input: '{'
  mods.gregtech.materialEvent {
                              ^
brachy84 commented 1 week ago

Groovy issue. Not really anything i can do about it

brachy84 commented 1 week ago

Took a deeper look and can confirm that fixing this issue might be impossible without fundamentaly changing the parser