RFR(XS): 8234011: (zipfs) Memory leak in ZipFileSystem.releaseDeflater()

Simonis, Volker simonisv at amazon.de
Sat Nov 16 08:03:37 UTC 2019


Am 15.11.2019 17:57 schrieb Lance Andersen <lance.andersen at oracle.com>:
>
> Hi Volker
>
> Lines 54.55  ArrayList<XX> foo = …
>
> This could be changed to List<XX> foo = …  if you wanted
>

Done and pushed.

Thanks for your patience,
Volker

> otherwise  I think you are good to go
>>
>> On Nov 15, 2019, at 10:41 AM, Langer, Christoph < christoph.langer at sap.com> wrote:
>>
>> Hi Volker,
>>
>>>> Please remove the "import java.nio.file.Files;" statement before pushing.
>>>>
>>>
>>> It's needed for "Files.deleteIfExists(zipFile)" in the finally block..
>>
>>
>> Oops, you're right. It popped up in my IDE because I tested what happened when the file remained in the directory and therefore I commented that block out.
>>
>> So, go for it ��
>>
>> Cheers
>> Christoph
>>
>
>
> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
> Oracle  Java Engineering
> 1 Network Drive
> Burlington, MA 01803
> Lance.Andersen at oracle.com
>
>
>




Amazon Development Center Germany GmbH
Krausenstr. 38
10117 Berlin
Geschaeftsfuehrung: Christian Schlaeger, Ralf Herbrich
Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B
Sitz: Berlin
Ust-ID: DE 289 237 879




More information about the core-libs-dev mailing list