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

[Roger Riggs]

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/