Thanks for the cleanup. +1.

Thanks,
Jay

> On 29-Jul-2020, at 2:28 PM, Sergey Bylokhov <sergey.bylok...@oracle.com> 
> wrote:
> 
> Hello.
> Please review the fix for jdk/client.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8250755
> Fix: http://cr.openjdk.java.net/~serb/8250755/webrev.00
> 
> The test in question creates two temp files. One of them properly cleaned 
> after
> http://cr.openjdk.java.net/~jdv/8183349/webrev.02/test/javax/imageio/plugins/shared/CanWriteSequence.java.sdiff.html
> 
> But there is another one created by the ImageOutputStream, so the 
> ImageOutputStream should be closed
> to delete this temp cache file.
> 
> -- 
> Best regards, Sergey.

Reply via email to