RFR JDK-8187485: Update Zip implementation to use Cleaner, not finalizers
Roger Riggs
Roger.Riggs at Oracle.com
Wed Dec 6 22:45:26 UTC 2017
Hi Sherman,
Looking good.
Deflater(573) and Inflater(398), and ZipFile(890):
For consistency, where is says "phantom-reachable" in the finalize
methods it can say "unreachable".
ZStreamRef:
- line 92: FinalizableZStreamRef - owner should be final
Regards, Roger
On 12/4/2017 6:14 PM, Xueming Shen wrote:
> Hi
>
> Please review the revision to the change for
>
> JDK-8187485: Update Zip implementation to use Cleaner, not finalizers
>
> For compatibility with previous jdk releases, in this proposed
> revision the corresponding
> close()/end() methods will be called when the
> ZipFile/Inflater/Deflater object has become
> unreachable, if the ZipFile/Inflater/Deflater has been subclassed and
> its close()/end() method
> has been overridden. In this case, the finalization mechanism is used
> to clean up the underlying
> resources. If the ZipFile/Inflater/Deflater has not been subclassed
> OR the corresponding
> close()/end() method has not been overridden, then the Cleaner
> mechanism is used to do
> the cleanup.
>
>
> issue: https://bugs.openjdk.java.net/browse/JDK-8185582
> webrev: http://cr.openjdk.java.net/~sherman/8185582/webrev
> csr: https://bugs.openjdk.java.net/browse/JDK-8187485
>
> Thanks,
> Sherman
More information about the core-libs-dev
mailing list