Skip to content

Clean up ATN serialization: rm UUID and shifting by value of 2 #3515

@parrt

Description

@parrt
  • I think we don't need the UUID in the serialization, since it has not changed in a decade. We can bump the version number and remove the UU ID
  • I did some tests and there seems to be no reason to shift the values in the serialized ATN by 2 for the purposes of improving the UTF-8 encoding for the Java target.

If you guys agree, we can make this small change for cleanup purposes. I'm happy to do it if you guys don't want to. The second fix will require changes to each target but it's trivial to fix.

Metadata

Metadata

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions