tmf.ui: make Comparator in Symbol Manager use the correct size.

Now compares integers as the priority is an integer.

Change-Id: I8bddbc8ff46cef34a7e12ee2bde69776a01226a8
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/78810
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

diff --git a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/symbols/SymbolProviderManager.java b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/symbols/SymbolProviderManager.java
index 25d9f29d049a58e68115e2cae1a4610a7803e44e..c749992f2c6e3f0b1e2a1e44eb11d4ce398c1a3e 100644 (file)
--- a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/symbols/SymbolProviderManager.java
+++ b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/symbols/SymbolProviderManager.java
@@ -96,9 +96,9 @@ public final class SymbolProviderManager {
              * Those with a higher priority need to be on top
              *
              * Note: we cannot simply sort by negative priority because
-             * (-Long.MIN_VAL) == Long.MIN_VAL
+             * (-Integer.MIN_VAL) == Integer.MIN_VAL
              */
-            providers.sort(Comparator.<SymbolProviderFactoryWrapper> comparingLong(o -> o.priority).reversed());
+            providers.sort(Comparator.<SymbolProviderFactoryWrapper> comparingInt(o -> o.priority).reversed());
             INSTANCE = new SymbolProviderManager(providers);
         }
         return INSTANCE;