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