On Wed, 22 Sep 2021 20:39:51 GMT, Andrey Turbanov <github.com+741251+turban...@openjdk.org> wrote:
> Cleanup places, where String.compareTo is used to check String's equality. > Instead String.equals or switch expression could be used. They are faster and > code is cleaner. src/java.management/share/classes/javax/management/BinaryRelQueryExp.java line 180: > 178: return sval1.compareTo(sval2) <= 0; > 179: case Query.EQ: > 180: return sval1.equals(sval2); I'm not so sure I agree this is better. Using `compareTo` keeps it consistent with the surrounding code. ------------- PR: https://git.openjdk.java.net/jdk/pull/5640
