On 14/12/2017 11:07, Claes Redestad wrote:
Hi,
my previous fix failed due to use of non-static inner classes which
kept the cleanable objects around. Also, Sherman suggested a more
thorough fix to Inflater/Deflater after I had already pushed.
Webrev: http://cr.openjdk.java.net/~redestad/8193507/open.00/
Verified all java/util/zip tests pass this time.
The changes means some code duplication but it's not too bad. Can the
ZStreamRef(long) constructor go away so that we don't need to think
about creating a ZStreamRef without a cleanable. Also can the fields in
InflaterCleanupAction be final.
-Alan
