Please Review fix for reduced value parser 8024076

Xueming Shen xueming.shen at oracle.com
Wed Oct 2 16:54:59 UTC 2013


Should move the static field BASE_DATE into ReducePrinterParser?
Logically (and for performance, if it matters at all) RPP appears to
be a better place for this constant.

The rest looks fine.

-Sherman

On 10/02/2013 08:19 AM, roger riggs wrote:
> Please review this fix for parsing two digit years in an Chronology.
>
> The webrev includes Stephen's proposed alternate method that provides
> a ChronoLocalDate as the base date.
>
> http://cr.openjdk.java.net/~rriggs/webrev-two-digit-8024076/
>
> Thanks, Roger
>
> p.s. the design issue raised in the comments has been filed as [2] : 8025828
>
> [1]https://bugs.openjdk.java.net/browse/JDK-8024076
> [2]https://bugs.openjdk.java.net/browse/JDK-8025828
>
>




More information about the core-libs-dev mailing list