Fix issue 864

MaxGreil · MaxGreil · commit 6e3627f13fcc · 2021-03-12T20:14:10.000+01:00
diff --git a/biojava-structure/src/main/java/org/biojava/nbio/structure/align/pairwise/AltAligComparator.java b/biojava-structure/src/main/java/org/biojava/nbio/structure/align/pairwise/AltAligComparator.java
@@ -48,20 +48,15 @@ public int compare(AlternativeAlignment a, AlternativeAlignment b) {
 
 		if ( s1 > s2)
 			return 1;
-		if ( s1 < s2)
+		else if ( s1 < s2)
 			return -1;
+		else {
+			// seem to have the same length
+			double rms1 = a.getRmsd();
+			double rms2 = b.getRmsd();
+			return Double.compare(rms1, rms2);
+		}
 
-		// seem to have the same length
-
-		double rms1 = a.getRmsd();
-		double rms2 = b.getRmsd();
-
-		if ( rms1 < rms2)
-			return 1;
-		if ( rms1 < rms2)
-			return -1;
-
-		return 0;
 	}
 
 
