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

Deperected api #98

Open
wants to merge 11 commits into
base: gradle-build-java-11
Choose a base branch
from

Conversation

Ao-senXiong
Copy link

Merge after #96

@sohah
Copy link

sohah commented Jun 23, 2024

@Ao-senXiong , thank you for your contributions. I have a few comments on this pull request:

  1. I see warnings from deprecated Gradle API. I think you can safely remove jcenter(). I also can see a reference to JavaPluginConvention which is also producing a couple of warnings. Can you investigate where this is being used?
  2. for SymbolicStringBuilder, can you please explain why you need java.lang.ref.Cleaner for Java 11 support?

@Ao-senXiong
Copy link
Author

1, I think we need to investigate on gradle 9.0 deprecation in another PR. This PR is more on Java API deprecation. For your question 2 for SymbolicStringBuilder, can you please explain why you need java.lang.ref.Cleaner for Java 11 support?, because _finalize method is deprecated since Java 9. So it is need to use the cleanner.

@sohah
Copy link

sohah commented Aug 11, 2024

@Ao-senXiong

  1. it would be nice to fix these gradle warning if possible, you have already done some of that, right?
  2. the usage of cleaner needs some fixes. I think you should capture the Cleanable from the register method, then invoke cleanable.clean() in the place where the _finalize() is right now. That way you've actually invoked the cleaning whenever the _finalize is invoked

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.

2 participants