From 875c779da5468c43eca05bb7eb1477836f6e076e Mon Sep 17 00:00:00 2001 From: gmungoc Date: Wed, 27 Jun 2018 15:05:02 +0100 Subject: [PATCH] JAL-3032 explicit Integer.intValue() conversion for safety --- src/jalview/util/QuickSort.java | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/src/jalview/util/QuickSort.java b/src/jalview/util/QuickSort.java index 0c0fabf..6901e24 100755 --- a/src/jalview/util/QuickSort.java +++ b/src/jalview/util/QuickSort.java @@ -51,8 +51,9 @@ public class QuickSort @Override public int compare(Integer o1, Integer o2) { - return ascending ? Float.compare(values[o1], values[o2]) - : Float.compare(values[o2], values[o1]); + return ascending + ? Float.compare(values[o1.intValue()], values[o2.intValue()]) + : Float.compare(values[o2.intValue()], values[o1.intValue()]); } } @@ -77,11 +78,11 @@ public class QuickSort { if (ascending) { - return Double.compare(values[o1], values[o2]); + return Double.compare(values[o1.intValue()], values[o2.intValue()]); } else { - return Double.compare(values[o2], values[o1]); + return Double.compare(values[o2.intValue()], values[o1.intValue()]); } } } @@ -105,8 +106,11 @@ public class QuickSort @Override public int compare(Integer o1, Integer o2) { - return ascending ? Integer.compare(values[o1], values[o2]) - : Integer.compare(values[o2], values[o1]); + return ascending + ? Integer.compare(values[o1.intValue()], + values[o2.intValue()]) + : Integer.compare(values[o2.intValue()], + values[o1.intValue()]); } } @@ -129,8 +133,9 @@ public class QuickSort @Override public int compare(Integer o1, Integer o2) { - return ascending ? values[o1].compareTo(values[o2]) - : values[o2].compareTo(values[o1]); + return ascending + ? values[o1.intValue()].compareTo(values[o2.intValue()]) + : values[o2.intValue()].compareTo(values[o1.intValue()]); } } -- 1.7.10.2