JDK 9 RFR of 8154183: (spec) {Data, }InputStream.read(byte[], int, int) - spec of offset argument confusing

[Roger Riggs]

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.Riggs at Oracle.com 
> <mailto:Roger.Riggs at 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}*.
>