Hi Chris, Thanks for the review. Here is a revised version, thanks to a comment from Daniel, which I think might be better:
Thanks, Brian --- a/src/java.base/share/classes/java/io/FileOutputStream.java +++ b/src/java.base/share/classes/java/io/FileOutputStream.java @@ -91,6 +91,10 @@ * If the file exists but is a directory rather than a regular file, does * not exist but cannot be created, or cannot be opened for any other * reason then a <code>FileNotFoundException</code> is thrown. + * <p> + * Invoking this constructor with the parameter {@code name} is equivalent + * to invoking {@link #FileOutputStream(String,boolean) + * new FileOutputStream(name, false)}. * * @param name the system-dependent filename * @exception FileNotFoundException if the file exists but is a directory On May 16, 2017, at 1:05 AM, Chris Hegarty <chris.hega...@oracle.com> wrote: > Looks good Brian. > > -Chris. > >> On 16 May 2017, at 02:27, Brian Burkhalter <brian.burkhal...@oracle.com> >> wrote: >> >> Please review at your convenience. >> >> Issue: https://bugs.openjdk.java.net/browse/JDK-8180353 >> Patch: [1] >> >> Thanks, >> >> Brian >> >> [1] Hg diff >> >> --- a/src/java.base/share/classes/java/io/FileOutputStream.java >> +++ b/src/java.base/share/classes/java/io/FileOutputStream.java >> @@ -91,6 +91,12 @@ >> * If the file exists but is a directory rather than a regular file, does >> * not exist but cannot be created, or cannot be opened for any other >> * reason then a <code>FileNotFoundException</code> is thrown. >> + * <p> >> + * Invoking this constructor with the parameter {@code name} is >> equivalent >> + * to invoking the constructor {@link #FileOutputStream(String,boolean) >> + * FileOutputStream(name,append)} with the same {@code String} parameter >> + * {@code name} and the {@code boolean} parameter {@code append} equal >> to >> + * {@code false}. >> * >> * @param name the system-dependent filename >> * @exception FileNotFoundException if the file exists but is a >> directory >