On Tue, 1 Sep 2026 22:20:31 GMT, Jose Pereda <[email protected]> wrote:

>> this is taken care of in L56
>
> 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).

>> L56
>
> 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

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2280#discussion_r3909006811
PR Review Comment: https://git.openjdk.org/jfx/pull/2280#discussion_r3909008807

Reply via email to