On Thu, 23 Sep 2021 02:52:11 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> I agree with Chris on this.
>> Fix in the other file looks okay to me.
>
> I look at it this way, the code should really (IMHO) have been written as:
>
> int res = sval1.compareTo(sval2);
> switch(relOp) {
> case Query.GT: return res > 0;
> case ...
> }
>
> and so the use of equals() instead would never have arisen. :)
Yep, that works for me.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5640
