--- old/src/java.base/share/classes/java/util/zip/Deflater.java 2017-11-01 12:54:18.996179743 +0100 +++ new/src/java.base/share/classes/java/util/zip/Deflater.java 2017-11-01 12:54:18.870181889 +0100 @@ -67,6 +67,20 @@ * } * * + *

+ * @apiNote + * In earlier versions the {@link Object#finalize} method was overridden and + * specified to call the {@code end} method to close the {@code deflater} and + * release the resource when the instance becomes unreachable. + * The {@code finalize} method is no longer defined. The recommended cleanup + * for compressor is to explicitly call {@code end} method when it is no + * longer in use. + * + * @implNote + * The resource of the compressor will be released when the instance becomes + * phantom-reachable, if the {@code end} is not invoked explicitly. + *

+ * * @see Inflater * @author David Connelly * @since 1.1 @@ -169,7 +183,9 @@ public Deflater(int level, boolean nowrap) { this.level = level; this.strategy = DEFAULT_STRATEGY; - this.zsRef = new ZStreamRef(init(level, DEFAULT_STRATEGY, nowrap)); + this.zsRef = new ZStreamRef(this, + () -> init(level, DEFAULT_STRATEGY, nowrap), + Deflater::end); } /** @@ -534,39 +550,18 @@ /** * Closes the compressor and discards any unprocessed input. + * * This method should be called when the compressor is no longer - * being used, but will also be called automatically by the - * finalize() method. Once this method is called, the behavior - * of the Deflater object is undefined. + * being used. Once this method is called, the behavior of the + * Deflater object is undefined. */ public void end() { synchronized (zsRef) { - long addr = zsRef.address(); - zsRef.clear(); - if (addr != 0) { - end(addr); - buf = null; - } + zsRef.clean(); + buf = null; } } - /** - * Closes the compressor when garbage is collected. - * - * @deprecated The {@code finalize} method has been deprecated. - * Subclasses that override {@code finalize} in order to perform cleanup - * should be modified to use alternative cleanup mechanisms and - * to remove the overriding {@code finalize} method. - * When overriding the {@code finalize} method, its implementation must explicitly - * ensure that {@code super.finalize()} is invoked as described in {@link Object#finalize}. - * See the specification for {@link Object#finalize()} for further - * information about migration options. - */ - @Deprecated(since="9") - protected void finalize() { - end(); - } - private void ensureOpen() { assert Thread.holdsLock(zsRef); if (zsRef.address() == 0)