From: Matthew Khouzam <matthew.khouzam@ericsson.com>
Date: Wed, 10 Aug 2016 21:45:23 +0000 (-0400)
Subject: tmf.ui: make Comparator in Symbol Manager use the correct size.
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=4c089034b71132abbb3f5b55daf3539ba05a7180;p=deliverable%2Ftracecompass.git

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 25d9f29d04..c749992f2c 100644
--- 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;