honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-03-09 14:34:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

don't use substraction for comparison if datatypes can overflow

Browse Source
This commit is contained in:
Simon Willnauer 2013-02-07 12:06:56 +01:00
parent f97021b165
commit 033d6e4306
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/GeoDistanceComparator.java
View File

@ -105,7 +105,13 @@ public class GeoDistanceComparator extends FieldComparator<Double> {
} else {
distance1 = fixedSourceDistance.calculate(geoPoint.lat(), geoPoint.lon());
}
return (int) (distance1 - distance2);
if (distance1 < distance2) {
return -1;
} else if (distance1 == distance2) {
return 0;
} else {
return 1;
}
}
@Override