On Mon, 2 Nov 2020 17:08:10 GMT, Stephen Colebourne <scolebou...@openjdk.org> 
wrote:

>> src/java.base/share/classes/java/time/format/Parsed.java line 497:
>> 
>>> 495:                         AMPM_OF_DAY.checkValidValue(ap);
>>> 496:                     }
>>> 497:                     updateCheckDayPeriodConflict(AMPM_OF_DAY, midpoint 
>>> / 720);
>> 
>> No need to put `AMPM_OF_DAY` back in here because you've already resolved it 
>> to `HOUR_OF_DAY` and `MINUTE_OF_HOUR`. There probably does need to be 
>> validation to check that the day period agrees with the AM/PM value.
>
> Line can still be removed AFAICT

Fixed.

-------------

PR: https://git.openjdk.java.net/jdk/pull/938

Reply via email to