On Mon, 3 Feb 2025 18:58:57 GMT, Justin Lu <j...@openjdk.org> wrote:

>> Please review this PR which improves the performance of cut-over date 
>> checking when the user supplies a properties override via the 
>> `java.util.currency.data` sys prop. Replacing the `SimpleDateFormat` with a 
>> _java.time_ alternative has better performance. It should be noted that this 
>> method is only entered when the string `s` is confirmed to adhere to the 
>> format: `yyyy-MM-ddTHH:mm:ss`.
>> 
>> An alternative is using `LocalDateTime.of(..)` and extracting the date/time 
>> values ourselves from `s` with the known positions which is considerably 
>> faster but not as concise.
>
> Justin Lu has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   reflect review

Marked as reviewed by aturbanov (Committer).

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

PR Review: https://git.openjdk.org/jdk/pull/23374#pullrequestreview-2590758515

Reply via email to