tmf: Use prioritized List instead of Set in XmlStateProvider

This allows to have a kind of priority between the event handlers. It
helps to improve dependency between the event handlers. Now, when
receiving an event E, it is possible that the result of a handler X
depends of the change made by a handler Y on the same event.

Change-Id: I33592e907d4c2f024a46cc365f0bbe796e940049
Signed-off-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/61672
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

diff --git a/tmf/org.eclipse.tracecompass.tmf.analysis.xml.core/src/org/eclipse/tracecompass/tmf/analysis/xml/core/stateprovider/XmlStateProvider.java b/tmf/org.eclipse.tracecompass.tmf.analysis.xml.core/src/org/eclipse/tracecompass/tmf/analysis/xml/core/stateprovider/XmlStateProvider.java
index b99a424a1f1878465b104551ee16f7b116b27854..ad73371c8523c9edc60121bf2253462d94ccf48b 100644 (file)
--- a/tmf/org.eclipse.tracecompass.tmf.analysis.xml.core/src/org/eclipse/tracecompass/tmf/analysis/xml/core/stateprovider/XmlStateProvider.java
+++ b/tmf/org.eclipse.tracecompass.tmf.analysis.xml.core/src/org/eclipse/tracecompass/tmf/analysis/xml/core/stateprovider/XmlStateProvider.java
@@ -12,6 +12,7 @@
 
 package org.eclipse.tracecompass.tmf.analysis.xml.core.stateprovider;
 
+import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.HashSet;
@@ -46,7 +47,7 @@ public class XmlStateProvider extends AbstractTmfStateProvider implements IXmlSt
     @NonNull private final String fStateId;
 
     /** List of all Event Handlers */
-    private final Set<TmfXmlEventHandler> fEventHandlers = new HashSet<>();
+    private final List<TmfXmlEventHandler> fEventHandlers = new ArrayList<>();
 
     /** List of all Locations */
     private final Set<TmfXmlLocation> fLocations;