tmf.core: Throw exception when no state system assigned

Instead of just logging an error that will happen for each event and
willl be very verbose, it should throw an exception.

Change-Id: Iaedcf461ffa5e8d9ac8abd4f11e98e9f70f6259e
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/84035
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

diff --git a/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/statesystem/AbstractTmfStateProvider.java b/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/statesystem/AbstractTmfStateProvider.java
index f9d74f948e4704904266086778978ae54502c6be..1dec0e0047e500963b13b6fe5dabd27b4222a30a 100644 (file)
--- a/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/statesystem/AbstractTmfStateProvider.java
+++ b/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/statesystem/AbstractTmfStateProvider.java
@@ -126,8 +126,7 @@ public abstract class AbstractTmfStateProvider implements ITmfStateProvider {
     public void processEvent(ITmfEvent event) {
         /* Make sure the target state system has been assigned */
         if (!fStateSystemAssigned) {
-            Activator.logError("Cannot process event without a target state system"); //$NON-NLS-1$
-            return;
+            throw new IllegalStateException("Cannot process event without a target state system. ID: " + getClass().getSimpleName()); //$NON-NLS-1$
         }
 
         /* Insert the event we're received into the events queue */