Skip to content
New issue

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

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

Already on GitHub? # to your account

[WINDUPRULE-1012] Add rules for the future deprecation of the finalize() methods #1009

Merged
merged 2 commits into from
Jul 25, 2023

Conversation

jmle
Copy link
Contributor

@jmle jmle commented Jul 24, 2023

The finalize() usage search has been implemented as a {*}.finalize() javaclass condition to avoid creating a where clause with many OR'd conditions inside.

See Jira ticket.

@PhilipCattanach
Copy link
Contributor

@jmle The rules worked fine.
Looking at the https://openjdk.org/jeps/421
This change was introduced in OpenJDK18
So I think the source folder should be openjdk17 not openjdk18
If you can rename that folder then we can get it merged.

@PhilipCattanach
Copy link
Contributor

All good - thanks @jmle

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants