[threeten-dev] looks a bug for JapaneseChronology

[Masayoshi Okutsu]

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
>
>
>
>
>