X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Futil%2FMapList.java;h=4658724e0f58cbda48e61745e9263109a7e59fc6;hb=5b7c6735d44fbd8d2c3fd36ede2ae7520fc16b7e;hp=dc5bee876134fee1ea42afc4a767a51c90fe893f;hpb=34abca821ae5c3f9220bfc246c3055e3ba9f39c3;p=jalview.git diff --git a/src/jalview/util/MapList.java b/src/jalview/util/MapList.java index dc5bee8..4658724 100644 --- a/src/jalview/util/MapList.java +++ b/src/jalview/util/MapList.java @@ -103,10 +103,9 @@ public class MapList { return false; } - return Arrays - .deepEquals(fromShifts.toArray(), obj.fromShifts.toArray()) - && Arrays - .deepEquals(toShifts.toArray(), obj.toShifts.toArray()); + return Arrays.deepEquals(fromShifts.toArray(), obj.fromShifts.toArray()) + && Arrays.deepEquals(toShifts.toArray(), + obj.toShifts.toArray()); } /** @@ -342,7 +341,8 @@ public class MapList */ public static List coalesceRanges(final List ranges) { - if (ranges == null || ranges.size() < 2) { + if (ranges == null || ranges.size() < 2) + { return ranges; } @@ -353,7 +353,7 @@ public class MapList lastRange = new int[] { lastRange[0], lastRange[1] }; merged.add(lastRange); boolean first = true; - + for (final int[] range : ranges) { if (first) @@ -372,10 +372,12 @@ public class MapList * drop this range if it lies within the last range */ if ((lastDirection == 1 && range[0] >= lastRange[0] - && range[0] <= lastRange[1] && range[1] >= lastRange[0] && range[1] <= lastRange[1]) + && range[0] <= lastRange[1] && range[1] >= lastRange[0] + && range[1] <= lastRange[1]) || (lastDirection == -1 && range[0] <= lastRange[0] && range[0] >= lastRange[1] - && range[1] <= lastRange[0] && range[1] >= lastRange[1])) + && range[1] <= lastRange[0] + && range[1] >= lastRange[1])) { changed = true; continue; @@ -387,10 +389,13 @@ public class MapList * if next range is in the same direction as last and contiguous, * just update the end position of the last range */ - boolean sameDirection = range[1] == range[0] || direction == lastDirection; + boolean sameDirection = range[1] == range[0] + || direction == lastDirection; boolean extending = range[0] == lastRange[1] + lastDirection; - boolean overlapping = (lastDirection == 1 && range[0] >= lastRange[0] && range[0] <= lastRange[1]) - || (lastDirection == -1 && range[0] <= lastRange[0] && range[0] >= lastRange[1]); + boolean overlapping = (lastDirection == 1 && range[0] >= lastRange[0] + && range[0] <= lastRange[1]) + || (lastDirection == -1 && range[0] <= lastRange[0] + && range[0] >= lastRange[1]); if (sameDirection && (overlapping || extending)) { lastRange[1] = range[1]; @@ -404,7 +409,7 @@ public class MapList lastDirection = (range[1] == range[0]) ? lastDirection : direction; } } - + return changed ? merged : ranges; } @@ -959,15 +964,17 @@ public class MapList // TODO not used - remove? if (local) { - return ((getFromLowest() >= map.getFromLowest() && getFromHighest() <= map - .getFromHighest()) || (getFromLowest() <= map.getFromLowest() && getFromHighest() >= map - .getFromHighest())); + return ((getFromLowest() >= map.getFromLowest() + && getFromHighest() <= map.getFromHighest()) + || (getFromLowest() <= map.getFromLowest() + && getFromHighest() >= map.getFromHighest())); } else { - return ((getToLowest() >= map.getToLowest() && getToHighest() <= map - .getToHighest()) || (getToLowest() <= map.getToLowest() && getToHighest() >= map - .getToHighest())); + return ((getToLowest() >= map.getToLowest() + && getToHighest() <= map.getToHighest()) + || (getToLowest() <= map.getToLowest() + && getToHighest() >= map.getToHighest())); } }