diff options
author | Jon Bratseth <bratseth@verizonmedia.com> | 2019-09-17 14:40:43 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@verizonmedia.com> | 2019-09-17 14:40:43 +0200 |
commit | 7c4e91a3f4f2553dea73ed4e230bd74655c7736e (patch) | |
tree | 575e852394594bbb2a431bef568e6da7618b76f4 /container-search/src/main/java/com/yahoo/search/grouping/request/RawBuffer.java | |
parent | fb4b666e78d936d0f6aadcc4f74cf208e8056776 (diff) |
Add equals and cleanup
Diffstat (limited to 'container-search/src/main/java/com/yahoo/search/grouping/request/RawBuffer.java')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/grouping/request/RawBuffer.java | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/grouping/request/RawBuffer.java b/container-search/src/main/java/com/yahoo/search/grouping/request/RawBuffer.java index 9b18d7b374c..8af5d47335c 100644 --- a/container-search/src/main/java/com/yahoo/search/grouping/request/RawBuffer.java +++ b/container-search/src/main/java/com/yahoo/search/grouping/request/RawBuffer.java @@ -106,7 +106,7 @@ public class RawBuffer implements Comparable<RawBuffer>, Cloneable { return 1; } } - return (my.length < their.length ? -1 : (my.length > their.length ? 1 : 0)); + return Integer.compare(my.length, their.length); } @Override @@ -115,10 +115,10 @@ public class RawBuffer implements Comparable<RawBuffer>, Cloneable { } @Override - public boolean equals(Object rhs) { - if (rhs instanceof RawBuffer) { - return (compareTo((RawBuffer)rhs) == 0); - } - return false; + public boolean equals(Object other) { + if (other == this) return true; + if ( ! (other instanceof RawBuffer)) return false; + return (compareTo((RawBuffer)other) == 0); } + } |