[threeten-dev] looks a bug for JapaneseChronology

[Patrick Zhang]

The fix is ok.
And I have found another problem but it is test bug. I will submit one 
new changeset for test code.

Regards
Patrick


On 5/9/13 12:12 PM, Masayoshi Okutsu wrote:
> There was a regression bug in my previous fixes. I've just pushed the 
> fix. I've confirmed that the attached test program works fine.
>
> Thanks,
> Masayoshi
>
> On 5/8/2013 6:38 PM, Patrick Zhang wrote:
>> Hi Team,
>>
>> During test development, I have found the behavior of 
>> JapaneseChronology has been changed in this week.
>> I just do one simple transformation from JapaneseDate to LocalDate:
>>
>>     JapaneseDate jdate1 = 
>> JapaneseChronology.INSTANCE.dateYearDay(JapaneseEra.HEISEI, 8, 60);
>>     JapaneseDate jdate2 = 
>> JapaneseChronology.INSTANCE.date(JapaneseEra.HEISEI, 8, 2, 29);
>>     System.out.println(LocalDate.from(jdate1));
>>     System.out.println(LocalDate.from(jdate2));
>>
>> It works well when I use jdk8 latest promotion/nightly build.
>> ========
>> gtee at jsn-gtwy-1: ~ $ export 
>> JAVA_HOME=/net/sqenfs-1/export1/jdk/re/8/promoted/latest/binaries/linux-i586/
>> gtee at jsn-gtwy-1: ~ $ $JAVA_HOME/bin/javac Test.java
>> gtee at jsn-gtwy-1: ~ $ $JAVA_HOME/bin/java Test
>> 1996-02-29
>> 1996-02-29
>> 1996-02-29
>> 1996-02-29
>> gtee at jsn-gtwy-1: ~ $ export 
>> JAVA_HOME=/net/sqenfs-1/export1/jdk/re/8/nightly-tl/ws-b89_2013-05-07-1702_4319/build/linux-i586/j2sdk-image/
>> gtee at jsn-gtwy-1: ~ $ $JAVA_HOME/bin/javac Test.java
>> gtee at jsn-gtwy-1: ~ $ $JAVA_HOME/bin/java Test
>> 1996-02-29
>> 1996-02-29
>> 1996-02-29
>> 1996-02-29
>> ========
>>
>> But when I shift to my local build, which I just generate after 
>> pulling latest change, the output looks strange:
>> ========
>> gtee at jsn-gtwy-1: ~ $ export 
>> JAVA_HOME=/net/sqenfs-1/export1/comp/jsn/users/patrick/threeten/build/linux-x86-normal-server-release/images/j2sdk-image/
>> gtee at jsn-gtwy-1: ~ $ $JAVA_HOME/bin/javac Test.java
>> gtee at jsn-gtwy-1: ~ $ $JAVA_HOME/bin/java Test
>> 1996-03-07
>> 1996-02-29
>> 1996-03-07
>> 1996-02-29
>> =========
>>
>> Please help to verify it.
>>
>> FYI, I do not meet such problem on last week. I remembered I have 
>> submitted one webrev on 04/28.
>> http://cr.openjdk.java.net/~pzhang/JSR310/java/time/chrono/JapaneseChronology/webrev/ 
>> <http://cr.openjdk.java.net/%7Epzhang/JSR310/java/time/chrono/JapaneseChronology/webrev/> 
>>
>> According the attached log, all looks good.
>> http://cr.openjdk.java.net/~pzhang/JSR310/java/time/chrono/JapaneseChronology/TCKJapaneseChronology.jtr 
>> <http://cr.openjdk.java.net/%7Epzhang/JSR310/java/time/chrono/JapaneseChronology/TCKJapaneseChronology.jtr> 
>>
>>
>> I pull many changeset today and rebuild the whole repo. Then the 
>> problem occurs. Please help to verify it.
>>
>> Regards
>> Patrick
>>
>>
>>
>>
>>
>