fix MarkerPriorityField.compare() eclipse-platform#2041

eclipse-platform#2041
Christopher-Hermann · Jul 9, 2024 · 0933929 · 0933929
1 parent a724c75
commit 0933929
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/bundles/org.eclipse.ui.ide/src/org/eclipse/ui/internal/views/markers/MarkerComparator.java b/bundles/org.eclipse.ui.ide/src/org/eclipse/ui/internal/views/markers/MarkerComparator.java
@@ -100,17 +100,16 @@ public int compare(MarkerItem item1, MarkerItem item2) {
 	 * @return int
 	 */
 	public int compareFields(MarkerItem item0, MarkerItem item1) {
-		int value = 0;
 		for (MarkerField field : fields) {
-			value = field.compare(item0, item1);
+			int value = field.compare(item0, item1);
 			if (value != 0) {
 				if (descendingFields.contains(field)) {
 					value = -value;
 				}
-				break;
+				return value;
 			}
 		}
-		return value;
+		return 0;
 	}
 	/**
 	 * Comparator to compare the two MarkerEntry(s) by various fields

diff --git a/...les/org.eclipse.ui.ide/src/org/eclipse/ui/internal/views/markers/MarkerPriorityField.java b/...les/org.eclipse.ui.ide/src/org/eclipse/ui/internal/views/markers/MarkerPriorityField.java
@@ -101,7 +101,7 @@ public MarkerPriorityField() {
 	@Override
 	public int compare(MarkerItem item1, MarkerItem item2) {
 		return Integer.compare(item1.getAttributeValue(IMarker.PRIORITY, IMarker.PRIORITY_NORMAL),
-				item1.getAttributeValue(IMarker.PRIORITY, IMarker.PRIORITY_NORMAL));
+				item2.getAttributeValue(IMarker.PRIORITY, IMarker.PRIORITY_NORMAL));
 	}
 
 	@Override