HI Brian,
Looks good,
Thanks for making them consistent, Roger
On 4/18/2016 4:53 PM, Brian Burkhalter wrote:
A new patch generated using the latest version of webrev is here:
http://cr.openjdk.java.net/~bpb/8154183/webrev.01/
<http://cr.openjdk.java.net/%7Ebpb/8154183/webrev.01/>
I have addressed the various editorial comments except with respect to
capitalizing the first word after the exception name and ending the
@throws description with a period: I have made the new @throws clauses
consistent with other such clauses already present in the same method.
In some case this means starting with lower case, in others upper
case; likewise in some cases there are periods at the end, in other
cases not.
Thanks,
Brian
On Apr 18, 2016, at 12:45 PM, Roger Riggs <roger.ri...@oracle.com
<mailto:roger.ri...@oracle.com>> wrote:
- In the new @throws NullPointerException and IndexOutOfBoundExceptions,
the first word after the exception should not be capitalized.
For example "if" instead of "If" makes it consistent with the
existing doc
ObjectInputStream.java: line 1012+
- usually the @throws description is not a complete sentence and does
not deserve a "."
per-file-consistency...
for example,
+ * @throws NullPointerException If {@code buf} is {@code null}*.
