RFR: 8044671: NPE from JapaneseEra when a new era is defined in calendar.properties

dmeetry degrave dmitry.degrave at oracle.com
Thu Jul 31 12:06:36 UTC 2014


Thanks for your review, Masayoshi!

I felt updating a copyright year can cause a conflict during importing a 
patch to another repositories (to previous releases, mostly).

Do we have a common rule for updating copyrights ?

thanks,
dmeetry

On 07/31/2014 03:27 AM, Masayoshi Okutsu wrote:
> The copyright year range should be updated. Otherwise, the fix looks
> good to me.
>
> Masayoshi
>
> On 7/23/2014 11:51 PM, dmeetry degrave wrote:
>> Hello,
>>
>> Please review a simple fix (jdk 8 and 9) for
>>
>> 8044671: NPE from JapaneseEra when a new era is defined in
>> calendar.properties
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8044671
>> fix: http://cr.openjdk.java.net/~dmeetry/8044671/webrev.01
>>
>> No automated regression test, unfortunately, it would require to
>> modify "private static final Era[] eras;" in
>> java/util/JapaneseImperialCalendar.java, though it's easy to test it
>> by calling 'JapaneseDate date = JapaneseDate.of(2100,1,1);' with a new
>> era added via JAVA_HOME/jre/lib/calendars.properties (test and
>> calendars.properties are attached to the bug).
>>
>> thanks,
>> dmeetry
>



More information about the core-libs-dev mailing list