RFR: 8225763: Inflater and Deflater should implement AutoCloseable [v4]

[Jaikiran Pai]

On Tue, 26 Nov 2024 17:36:40 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Done.
>
> Inflater/Deflater date from JDK 1.1 so I think there is some risk to adding a final no-arg close method. I understand that a corpus analysis has been done, and the intention of making it final is for subclasses to know which method to override, but it doesn't take from the possibility that it might break subclasses developed years ago. If it goes ahead as reviewed, then we'll to do some outreach and be prepared to back out this change in the event  that breakage is reported before GA.

Hello Alan, 

> If it goes ahead as reviewed, then we'll to do some outreach and be prepared to back out this change in the event that breakage is reported before GA.

I agree that if the `final close()` causes issues, then backing this change out and reconsidering a non-final `close()` would be the right thing to do. Once/if this gets integrated into 24, I will work with the outreach team to have this change noted in their EA mails.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/19675#discussion_r1860008376