fix a few things

Author: SylvainJuge

instrumentation/jaxrs/jaxrs-1.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxrs/v1_0/JaxrsAnnotationsInstrumentation.java

@@ -17,6 +17,7 @@
 import static net.bytebuddy.matcher.ElementMatchers.namedOneOf;
 import static net.bytebuddy.matcher.ElementMatchers.not;
 
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.Scope;
 import io.opentelemetry.instrumentation.api.semconv.http.HttpServerRoute;
@@ -78,6 +79,7 @@ public AdviceScope(CallDepth callDepth) {
         this.callDepth = callDepth;
       }
 
+      @CanIgnoreReturnValue
       public AdviceScope enter(Class<?> type, Method method) {
         if (callDepth.getAndIncrement() > 0) {
           return this;

instrumentation/jaxrs/jaxrs-2.0/jaxrs-2.0-annotations/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxrs/v2_0/JaxrsAnnotationsInstrumentation.java

@@ -16,6 +16,7 @@
 import static net.bytebuddy.matcher.ElementMatchers.namedOneOf;
 import static net.bytebuddy.matcher.ElementMatchers.not;
 
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.Scope;
 import io.opentelemetry.instrumentation.api.semconv.http.HttpServerRoute;
@@ -82,6 +83,7 @@ public AdviceScope(CallDepth callDepth) {
         this.callDepth = callDepth;
       }
 
+      @CanIgnoreReturnValue
       public AdviceScope enter(Object[] args, Object target, Method method) {
         if (callDepth.getAndIncrement() > 0) {
           return this;
@@ -128,6 +130,7 @@ public AdviceScope enter(Object[] args, Object target, Method method) {
       }
 
       @Nullable
+      @CanIgnoreReturnValue
       public Object exit(@Nullable Throwable throwable, @Nullable Object returnValue) {
 
         if (callDepth.decrementAndGet() > 0) {

instrumentation/jaxrs/jaxrs-2.0/jaxrs-2.0-jersey-2.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxrs/v2_0/JerseyServletContainerInstrumentation.java

@@ -15,6 +15,7 @@
 import io.opentelemetry.javaagent.bootstrap.jaxrs.JaxrsContextPath;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeInstrumentation;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeTransformer;
+import javax.annotation.Nullable;
 import javax.servlet.http.HttpServletRequest;
 import net.bytebuddy.asm.Advice;
 import net.bytebuddy.description.type.TypeDescription;

instrumentation/jaxrs/jaxrs-3.0/jaxrs-3.0-annotations/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxrs/v3_0/JaxrsAnnotationsInstrumentation.java

@@ -16,6 +16,7 @@
 import static net.bytebuddy.matcher.ElementMatchers.namedOneOf;
 import static net.bytebuddy.matcher.ElementMatchers.not;
 
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
 import io.opentelemetry.context.Context;
 import io.opentelemetry.context.Scope;
 import io.opentelemetry.instrumentation.api.semconv.http.HttpServerRoute;
@@ -82,6 +83,7 @@ public AdviceScope(CallDepth callDepth) {
         this.callDepth = callDepth;
       }
 
+      @CanIgnoreReturnValue
       public AdviceScope enter(Class<?> type, Method method, Object[] args) {
         if (callDepth.getAndIncrement() > 0) {
           return this;
@@ -128,6 +130,7 @@ public AdviceScope enter(Class<?> type, Method method, Object[] args) {
       }
 
       @Nullable
+      @CanIgnoreReturnValue
       public Object exit(@Nullable Object returnValue, @Nullable Throwable throwable) {
         if (callDepth.decrementAndGet() > 0) {
           return returnValue;