On Fri, 3 Apr 2026 18:00:48 GMT, Weijun Wang <[email protected]> wrote:

>> I wanted to keep the `date/time` here as it seemed to me to be easier to 
>> read than repeating `instant`. Just wanted to keep it in clarification.
>> 
>> If you still think it's better to make it `instant` - I don't mind changing
>
> I was thinking that back when `checkValidaty(Date)` was introduced, the 
> author used `date` to match the argument type, but then thought it's not 
> precise enough and added `time`. Now we don't have this inconvenience. Just 
> my guess.

Ok, will change in the next commit. Thank you

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30047#discussion_r3044365560

Reply via email to