RFR 8/9: 8068278 ArrayIndexOutOfBoundsException instead of DateTimeException in j.t.chrono.JapaneseChronology.eraOf()

Roger Riggs Roger.Riggs at Oracle.com
Fri Jan 30 22:33:45 UTC 2015


Hi Mandy,

Thanks for the review.

I wrote the test (and it passed) but since the JCK folks are providing 
the tests it seemed
undesirable to have duplicate tests.

Roger

On 1/30/2015 5:27 PM, Mandy Chung wrote:
> On 1/30/15 2:25 PM, Roger Riggs wrote:
>> Please review this correction of a JapaneseEra range check in java.time.
>> The error was discovered during development of additional conformance 
>> tests (to be delivered separately).
>>
>> Webrev:
>>    http://cr.openjdk.java.net/~rriggs//webrev-era-8068278
>>
>
> Looks fine to me.  Is it easy to write a regression test to go along 
> with this fix?
>
> Mandy
>
>> Issue:
>> <https://bugs.openjdk.java.net/browse/JDK-8068278> 8068278 
>> ArrayIndexOutOfBoundsException instead of DateTimeException in 
>> j.t.chrono.JapaneseChronology.eraOf()
>>
>> Thanks, Roger
>>
>




More information about the core-libs-dev mailing list