From: Matthew Khouzam <matthew.khouzam@ericsson.com>
Date: Sun, 20 Mar 2016 21:11:06 +0000 (-0400)
Subject: linux.core: Add messages to IllegalStateExceptions
X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=878d6a0e6ad1f56f2b60912bf03781f0471b284c;p=deliverable%2Ftracecompass.git

linux.core: Add messages to IllegalStateExceptions

Analysis modules now declare why they fail.

Change-Id: Idf4ec5ac0a0c7b96317512432ca8d9ab0ee4c61c
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/68875
Reviewed-by: Hudson CI
Reviewed-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
Tested-by: Jean-Christian Kouame <jean-christian.kouame@ericsson.com>
---

diff --git a/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/contextswitch/KernelContextSwitchAnalysis.java b/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/contextswitch/KernelContextSwitchAnalysis.java
index d616281130..7dc6b2ceaf 100644
--- a/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/contextswitch/KernelContextSwitchAnalysis.java
+++ b/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/contextswitch/KernelContextSwitchAnalysis.java
@@ -84,7 +84,7 @@ public class KernelContextSwitchAnalysis extends TmfStateSystemAnalysisModule {
 
         ITmfTrace trace = getTrace();
         if (trace == null) {
-            throw new IllegalStateException();
+            throw new IllegalStateException("Analysis requires a trace"); //$NON-NLS-1$
         }
         /*
          * This analysis depends on the LTTng kernel analysis, so it's added to
diff --git a/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/cpuusage/KernelCpuUsageAnalysis.java b/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/cpuusage/KernelCpuUsageAnalysis.java
index 98d9446d84..0fb4f3135d 100644
--- a/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/cpuusage/KernelCpuUsageAnalysis.java
+++ b/analysis/org.eclipse.tracecompass.analysis.os.linux.core/src/org/eclipse/tracecompass/analysis/os/linux/core/cpuusage/KernelCpuUsageAnalysis.java
@@ -106,7 +106,7 @@ public class KernelCpuUsageAnalysis extends TmfStateSystemAnalysisModule {
 
         ITmfTrace trace = getTrace();
         if (trace == null) {
-            throw new IllegalStateException();
+            throw new IllegalStateException("Analysis requires a trace"); //$NON-NLS-1$
         }
         /*
          * This analysis depends on the LTTng kernel analysis, so it's added to