[13] RFR: 8223773: DateTimeFormatter Fails to throw an Exception on Invalid CLOCK_HOUR_OF_AMPM and HOUR_OF_AMPM

Lance Andersen lance.andersen at oracle.com
Wed May 29 21:21:11 UTC 2019


Hi Naoto,

This is OK.

Could you please add an @summary as it is missing so that we are consistent with other tests.

No need to spin a webrev again though


Best
Lance
> On May 29, 2019, at 5:12 PM, naoto.sato at oracle.com wrote:
> 
> Hi,
> 
> Please review the fix for the following issue:
> 
> https://bugs.openjdk.java.net/browse/JDK-8223773
> 
> The proposed changeset is located at:
> 
> https://cr.openjdk.java.net/~naoto/8223773/webrev.00/
> 
> Checking the range of HourOfAmPm with the range of AmPmOfDay is apparently incorrect. Fixing it will change the behavior of parsing with SMART resolver style to throw a DateTimeParseException. This is a change in which an app will start to see an exception with incorrect data, but that is what should be in the first place.
> 
> Naoto

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>





More information about the core-libs-dev mailing list