On Tue, 29 Mar 2022 15:27:28 GMT, Claes Redestad <redes...@openjdk.org> wrote:

>> In `LocalDate::ofEpochDays` we validate the epoch day input, then we also 
>> validate the year derived from that value. This second validation is 
>> redundant since the minimum and maximum valid epoch day line up with the 
>> first and last day of the minimum and maximum valid year, respectively. This 
>> patch replace this redundant runtime validation with a test. 
>> 
>> This reduces code complexity (increasing chance for inlining to happen) and 
>> removes a couple of branches from generated code.
>
> Claes Redestad has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   typo: to -> of

Marked as reviewed by naoto (Reviewer).

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

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

Reply via email to