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