common: Annotate some Class.get*Name methods

This fixes a warning in AlgorithmManager. getName and getSimpleName
cannot return null whereas getCanonicalName can (anonymous, local
classes).

Change-Id: I2ee6ff668a93b47617389d2a3a25157af7184349
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/62077
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>

diff --git a/common/org.eclipse.tracecompass.common.core/annotations/java/lang/Class.eea b/common/org.eclipse.tracecompass.common.core/annotations/java/lang/Class.eea
index 1d432187d6a92fdd7f7ec10ce547162aec7263ca..29faa70b41c06f1275eca3ae22010c6c03050ffe 100644 (file)
--- a/common/org.eclipse.tracecompass.common.core/annotations/java/lang/Class.eea
+++ b/common/org.eclipse.tracecompass.common.core/annotations/java/lang/Class.eea
@@ -2,3 +2,12 @@ class java/lang/Class
 getAnnotation
  <A::Ljava/lang/annotation/Annotation;>(Ljava/lang/Class<TA;>;)TA;
  <A::Ljava/lang/annotation/Annotation;>(Ljava/lang/Class<TA;>;)T0A;
+getCanonicalName
+ ()Ljava/lang/String;
+ ()L0java/lang/String;
+getName
+ ()Ljava/lang/String;
+ ()L1java/lang/String;
+getSimpleName
+ ()Ljava/lang/String;
+ ()L1java/lang/String;

diff --git a/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/internal/tmf/core/request/TmfRequestExecutor.java b/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/internal/tmf/core/request/TmfRequestExecutor.java
index cba3cae6965df2a991b0021397a98bb3021eaeb3..4eda5c05e28a15ee01904f371d46ee6d0a4298e7 100644 (file)
--- a/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/internal/tmf/core/request/TmfRequestExecutor.java
+++ b/tmf/org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/internal/tmf/core/request/TmfRequestExecutor.java
@@ -14,6 +14,8 @@
 
 package org.eclipse.tracecompass.internal.tmf.core.request;
 
+import static org.eclipse.tracecompass.common.core.NonNullUtils.checkNotNull;
+
 import java.util.Queue;
 import java.util.Timer;
 import java.util.TimerTask;
@@ -75,7 +77,8 @@ public class TmfRequestExecutor implements Executor {
      * Default constructor
      */
     public TmfRequestExecutor() {
-        String canonicalName = fExecutor.getClass().getCanonicalName();
+        // We know the canonical name is not null because we use ExecutorService only
+        String canonicalName = checkNotNull(fExecutor.getClass().getCanonicalName());
         fExecutorName = canonicalName.substring(canonicalName.lastIndexOf('.') + 1);
         if (TmfCoreTracer.isComponentTraced()) {
             TmfCoreTracer.trace(fExecutor + " created"); //$NON-NLS-1$