On 12/29/2014 04:51 PM, Alan Bateman wrote:
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.
Thanks, Alan.
Peter
