Hi Brian,
Looks good.
A nit, In PrintStream.java 597 and 602 : put in the spaces after ",".
Thanks, Roger
On 7/23/19 2:40 PM, Brian Burkhalter wrote:
On Jul 23, 2019, at 10:09 AM, Brian Burkhalter <brian.burkhal...@oracle.com>
wrote:
You will want to add an @throws
Will do.
The implSpec
—————
* @implSpec
* The default implementation is equivalent to
* {@link java.io.FilterOutputStream#write(byte[],int,int)
* super.write(buf,0,buf.length)}.
*
——————
Not sure if “The default implementation” is the correct wording given it is not
a default method (not sure what the norm is for describing the implementation
in this case). Perhaps something like “This method is equivalent to calling ….”
I concur, that would be better.
Above changes made in [1].
Thanks,
Brian
[1] http://cr.openjdk.java.net/~bpb/8187898/webrev-alt.03/