diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-04-21 16:07:53 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-04-21 16:07:53 +0200 |
commit | 84eb3fdbd7531d87c09f2a7e25c94f7fa6cdd3e0 (patch) | |
tree | fe9d4f771705cb149194c8a7432235100d88590d /searchlib/src/main | |
parent | cf9e699bef6999602b7801e0d01cc322d034c10b (diff) |
Add ordering test for IntegerBucketResultNode and fix overflow bug.
Diffstat (limited to 'searchlib/src/main')
-rw-r--r-- | searchlib/src/main/java/com/yahoo/searchlib/expression/IntegerBucketResultNode.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/searchlib/src/main/java/com/yahoo/searchlib/expression/IntegerBucketResultNode.java b/searchlib/src/main/java/com/yahoo/searchlib/expression/IntegerBucketResultNode.java index 4bf33b15ffe..c5d66406fd5 100644 --- a/searchlib/src/main/java/com/yahoo/searchlib/expression/IntegerBucketResultNode.java +++ b/searchlib/src/main/java/com/yahoo/searchlib/expression/IntegerBucketResultNode.java @@ -81,11 +81,11 @@ public class IntegerBucketResultNode extends BucketResultNode { return (classId - rhs.getClassId()); } IntegerBucketResultNode b = (IntegerBucketResultNode)rhs; - long diff = from - b.from; - if (diff == 0) { - diff = to - b.to; - } - return ((diff == 0) ? 0 : ((diff < 0) ? -1 : 1)); + if (from < b.from) return -1; + if (from > b.from) return 1; + if (to < b.to) return -1; + if (to > b.to) return 1; + return 0; } @Override |