On Tue, 1 Aug 2023 11:25:49 GMT, Jose Pereda <jper...@openjdk.org> wrote:

>> Probably something to evaluate, but in a separate ticket
>
> I agree, this should probably be considered as well, as part of a separate 
> ticket. 
> 
> Note that, when the file already exists in the cache, the initial InputStream 
> object is used by the DigestInputStream, and it is manually closed and reread 
> again, so special care should be taken here.

Unless we suspect a problem, I don't see a need to do anything differently.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1188#discussion_r1283083999

Reply via email to