JDK 9 RFR of 8067669: Documentation for methods in Number incomplete regarding too large values.

Roger Riggs Roger.Riggs at Oracle.com
Thu Jan 29 21:07:23 UTC 2015


Hi Brian,

#1, The current webrev is fine as is.  (Perhaps with a 2015 copyright 
update).

Since the methods are abstract, the general description is sufficient and
the subclass would have more detail if specified.

Roger

On 1/29/2015 3:53 PM, Brian Burkhalter wrote:
> On Jan 19, 2015, at 12:32 AM, Andreas Lundblad <andreas.lundblad at oracle.com> wrote:
>
>>> http://cr.openjdk.java.net/~bpb/8067669/webrev.01/
>>>
>>> Note that the change at line 40 should be made even if the other diffs are rejected.
>> This patch is an improvement in my opinion since it does not indicate that any effort is made to round or truncate the number. The obvious place to look for further documentation would be in the implementing class. I am however still in favour of making this explicit by adding "The specific semantics of the conversion is defined by the subclass in question." which is also in line with the class level documentation.
> So at this point I think there are three options:
>
> 1) webrev.01 above
>
> 2) webrev.01 plus the statement
>
> 	"The specific semantics of the conversion is defined by the subclass in question.”
>
> added to the doc of each conversion method
>
> 3) resolve as “Not an Issue.”
>
> It would be good to decide on one of these and resolve this issue. I am inclined to either #1 or #3.
>
> Thanks,
>
> Brian




More information about the core-libs-dev mailing list