On Fri, 18 Aug 2023 23:14:20 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
> Revise some verbiage about the `RandomAccessFile(String,String)` constructor > so that the string name of a file is referred to as _pathname string_ for > consistency with `java.io.File`. src/java.base/share/classes/java/io/RandomAccessFile.java line 213: > 211: * @throws SecurityException if a security manager exists and > its > 212: * {@code checkRead} method denies read access to the > file > 213: * or the mode is {@code "rw"} and the security manager's GitHub UI isn't allowing me to add a review comment on line 195, so adding it here. Should line 195 be changed to say: also called with the pathname of the {@code file} argument to see if write access to the file... to match what it says a few lines above for the `checkRead` security check? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/15351#discussion_r1299313238