On Wed, 14 Oct 2020 10:48:55 GMT, Volker Simonis <simo...@openjdk.org> wrote:

>> test/jdk/java/util/zip/CopyZipFile.java line 104:
>> 
>>> 102:         // all these fields set to '-1'.
>>> 103:         InputStream is = new FileInputStream(ZIP_FILE);
>>> 104:         ZipInputStream zis = new ZipInputStream(is);
>> 
>> Any reason not to include the the ZipInputStream and InputStream in the try 
>> with Resources and the ZipFile below?  I
>> know these are nits, but just curious or was it an over site?
>> I think we are good otherwise
>
> No other reason except my laziness :)

fixed now

-------------

PR: https://git.openjdk.java.net/jdk/pull/520

Reply via email to