RFR:JDK-8032051:"ZonedDateTime" class "parse" method fails with short time zone offset ("+01")

Roger Riggs Roger.Riggs at Oracle.com
Mon Feb 22 21:22:25 UTC 2016


Hi Nadeesh,

Sorry for the delay.

DateTimeFrmatterBuilder.java: 3387:  avoid testing isStrict twice.
   - refactor the if's so there is an outer if for context.isStrict == 
false and
     inner if's  (or tertiary ?: assignment) for the replacement parseType.
    if (context.isStrict() == false) {
       if ( type == 6) {
         parseType = 10;
       } else if (type == 5) {
         parseType = 9;
       }
    }


Otherwise looks fine.

Thanks Roger



On 2/22/2016 3:50 AM, nadeesh tv wrote:
> Gentle Reminder
> On 2/12/2016 1:52 AM, nadeesh tv wrote:
>> Hi all,
>>
>> Please review a fix for
>>
>> Bug Id  https://bugs.openjdk.java.net/browse/JDK-8032051
>>
>> webrev http://cr.openjdk.java.net/~ntv/8032051/webrev.01/
>>
>




More information about the core-libs-dev mailing list