Niels Basjes
Niels Basjes
Thanks @michael-o !
Looks good to me.
@parrt FYI: The bug report https://github.com/antlr/antlr4/issues/4341 was only partially fixed by https://github.com/antlr/antlr4/pull/4394 . This pull request fixes the missing part for that issue.
Gentle ping to @parrt, this is ready for review.
Additional reproduction: Take the reproduction of https://github.com/antlr/antlr4/issues/4341 located here https://github.com/khmarbaise/issue-antlr-4.13.0 Clone this and change the version to 4.13.1 When you actually build this with Java 21 you'll see that the...
This is the code: https://github.com/nielsbasjes/yauaa/blob/main/analyzer/pom.xml#L419-L434 A scripted search and replace to fix this issue in the code generated by Antlr
[modbus-sunspec.zip](https://github.com/apache/plc4x/files/10420712/modbus-sunspec.zip)
For me this is not seen as a bug in your library but as a peculiar response of a real life system. Ill double check if your newer implementation changes...
Also; I realized a while ago that the only way any library can reliable read this device is when this library has a full schema of the device at hand....