8157677: Subclasses of Reader do not inherit the contents in the exception tag from the parent Reader class in the latest spec

[Roger Riggs]

Hi Pavel,

To keep it simple and clean, the change from @exception to @throws is 
not needed
and reduces the number of files changed.

The change in the parameter name is a bit unusual, typically they are 
not changed
but in this case the change is needed to make parameter name consistent with
the *inherited* specification.

The change of @exception to @throws can be done consistently across the 
file later.

$.02, Roger


On 5/24/16 10:24 AM, Pavel Rappo wrote:
> Thanks for looking at this!
>
>> On 24 May 2016, at 15:16, Aleksey Shipilev <aleksey.shipilev at oracle.com> wrote:
>>
>> *) CharArrayReader: code changes
> What's your concern with 'b'->'buf' rename? Access parameter's name through
> reflection or something else?
>
>> *) It's weird to see @exception and @throws side-by-side:
>>
>>       * @exception  IOException  If an I/O error occurs
>>       * @throws     IndexOutOfBoundsException {@inheritDoc}
>>
> I wanted to address this issue only, i.e. to be very focused.
>
>> Wouldn't it be better to raise the priority for the javadoc bug, and fix
>> it there?
> It doesn't mean we can't raise the priority, though it's not me to decide this.
> Especially, given there's a trivial workaround available.
>
>