On 29/12/2014 09:45, Peter Levart wrote:
Thanks for looking at this, Alan.
You're right about File.getCanonicalFile(). It already checks read
permission for a file. The additional explicit check is superfluous. I
have removed it.
With explicit check I wanted the API to behave uniformly regardless of
whether the returned stream is opened by getInputStream() call or an
already opened stream is just returned. getCannonicalFile() already
takes care of it. Here's the updated webrev:
http://cr.openjdk.java.net/~plevart/jdk9-dev/FileInputStreamPool.8047769/webrev.03/
Updated patch looks good to me.
-Alan.
