On 12/4/17 3:14 PM, Xueming Shen wrote:
issue: https://bugs.openjdk.java.net/browse/JDK-8185582 webrev: http://cr.openjdk.java.net/~sherman/8185582/webrev
ZStreamRef.java 79 static ZStreamRef get(Object owner, LongSupplier addr, LongConsumer end) { It may be better to have two factory methods that take the specific type (Deflater & Inflater) instead of this single method taking Object owner. ZipFile.java 701 // List of cached Inflater objects for decompression 702 Deque<Inflater> inflaterCache;should this be a volatile field?
TestCleaner.java 27 * @summary Check the resources of Inflater, Deflater and ZipFile are always 28 * cleaned/released when the instane is not unreachable typo: s/instane/instance and s/not unreachable/unreachable Otherwise looks good. Mandy
