Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suppress "this-escape" warnings from generated encoders and decoders #531

Closed
wants to merge 3 commits into from

Conversation

writeoncereadmany
Copy link

See #494

This is the simplest, least invasive solution to avoid warnings on generated classes.

…coders, required for them to compile without warnings in JDK21
…decoder, leaving ghosts references when parsing new message with the same decoder instance - issue 525."

This reverts commit 99f3d76.
@writeoncereadmany
Copy link
Author

We needed various other changes on the branch we use in production: I'll raise separate branches for the two features we want merged, the last commit also has an issue raised

Comment on lines 845 to +846

//this test was added for issue #525
//TODO this test was added to guide the bugfix

Check notice

Code scanning / CodeQL

Unread local variable Note test

Variable 'Decoder decoder' is never read.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant