On Tue, 1 Sep 2026 22:33:27 GMT, Andy Goryachev <[email protected]> wrote:

>> Right, but only because the order of calls is like this, but it that changes 
>> in the future, or there is another place that calls directly 
>> findHitMidpoint(), then there is no check in place.
>
> the order does not matter: sz >= 2 invariant is ensured in in the factory 
> method (before the constructor).

Right

>> In this case, sz > 1 is needed, L56 is just sz != 0 (and also, the comment 
>> about the order of calls)
>
> sz is always >= 2

right

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2280#discussion_r3912484405
PR Review Comment: https://git.openjdk.org/jfx/pull/2280#discussion_r3912567278

Reply via email to