[12] RFR: 8212941: Loosen the range of JapaneseEra
Naoto Sato
naoto.sato at oracle.com
Tue Oct 30 17:45:41 UTC 2018
Hi Chris,
Thank you for the review.
On 10/30/18 10:33 AM, Chris Hegarty wrote:
>
>> On 30 Oct 2018, at 17:03, Roger Riggs <Roger.Riggs at oracle.com> wrote:
>>
>> Hi Naoto,
>>
>> Looks fine. The wording should allow future Japanese era to be defined without
>> the timing being tightly coupled to java specification updates.
>
> +1
>
> Suggest: "The defined ** era’s {@link #getValue} ** is expected
> to have a consecutive integer associated with it.”
>
> I suspect that the wording here has deliberately chosen, but I
> wonder if it could be tightened a little?
>
> `values()` should return an array of eras whose values are in a
> contiguous range, no? If so, can this be specified?
Yes, both of those are valid points. I have already pushed the change,
as the CSR has been approved, I will consider modifying the wording
according to your suggestion in the future.
Naoto
>
> -Chris.
>
>
>> Thanks, Roger
>>
>> On 10/30/2018 12:29 PM, Naoto Sato wrote:
>>> Updated the webrev. Please review.
>>>
>>> http://cr.openjdk.java.net/~naoto/8212941/webrev.03/
>
More information about the core-libs-dev
mailing list