Some minor cleanups (#627)

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
open-telemetry · Oct 24, 2019 · c24e4a9 · c24e4a9
1 parent bcc6202
commit c24e4a9
Show file tree

Hide file tree

Showing 10 changed files with 7 additions and 12 deletions.
diff --git a/all/build.gradle b/all/build.gradle
@@ -42,7 +42,7 @@ javadoc {
     })
     for (subproject in subprojects_javadoc) {
         if (subproject == project) {
-            continue;
+            continue
         }
         source subproject.javadoc.source
         options.links subproject.javadoc.options.links.toArray(new String[0])

diff --git a/api/src/main/java/io/opentelemetry/distributedcontext/EmptyDistributedContext.java b/api/src/main/java/io/opentelemetry/distributedcontext/EmptyDistributedContext.java
@@ -26,7 +26,7 @@
  */
 @Immutable
 public class EmptyDistributedContext implements DistributedContext {
-  private static final Collection<Entry> EMPTY_COLLECTION = Collections.<Entry>emptyList();
+  private static final Collection<Entry> EMPTY_COLLECTION = Collections.emptyList();
 
   /**
    * Returns the single instance of the {@link EmptyDistributedContext} class.

diff --git a/api/src/main/java/io/opentelemetry/distributedcontext/EntryMetadata.java b/api/src/main/java/io/opentelemetry/distributedcontext/EntryMetadata.java
@@ -89,7 +89,7 @@ public enum EntryTtl {
 
     private final int hops;
 
-    private EntryTtl(int hops) {
+    EntryTtl(int hops) {
       this.hops = hops;
     }
 

diff --git a/api/src/main/java/io/opentelemetry/trace/Status.java b/api/src/main/java/io/opentelemetry/trace/Status.java
@@ -203,7 +203,7 @@ public enum CanonicalCode {
 
     private final int value;
 
-    private CanonicalCode(int value) {
+    CanonicalCode(int value) {
       this.value = value;
     }
 

diff --git a/api/src/main/java/io/opentelemetry/trace/TraceFlags.java b/api/src/main/java/io/opentelemetry/trace/TraceFlags.java
@@ -232,7 +232,6 @@ public Builder setIsSampled(boolean isSampled) {
         options = (byte) (options | IS_SAMPLED);
       } else {
         options = (byte) (options & ~IS_SAMPLED);
-        ;
       }
       return this;
     }

diff --git a/api/src/main/java/io/opentelemetry/trace/Tracestate.java b/api/src/main/java/io/opentelemetry/trace/Tracestate.java
@@ -131,7 +131,6 @@ private Builder(Tracestate parent) {
      * @return this.
      * @since 0.1.0
      */
-    @SuppressWarnings("nullness")
     public Builder set(String key, String value) {
       // Initially create the Entry to validate input.
       Entry entry = Entry.create(key, value);
@@ -158,7 +157,6 @@ public Builder set(String key, String value) {
      * @return this.
      * @since 0.1.0
      */
-    @SuppressWarnings("nullness")
     public Builder remove(String key) {
       Utils.checkNotNull(key, "key");
       if (entries == null) {

diff --git a/api/src/main/java/io/opentelemetry/trace/propagation/HttpTraceContext.java b/api/src/main/java/io/opentelemetry/trace/propagation/HttpTraceContext.java
@@ -146,8 +146,7 @@ public <C> void inject(SpanContext spanContext, C carrier, Setter<C> setter) {
         String listMember = listMembers[i];
         int index = listMember.indexOf(TRACESTATE_KEY_VALUE_DELIMITER);
         checkArgument(index != -1, "Invalid tracestate list-member format.");
-        tracestateBuilder.set(
-            listMember.substring(0, index), listMember.substring(index + 1, listMember.length()));
+        tracestateBuilder.set(listMember.substring(0, index), listMember.substring(index + 1));
       }
       return SpanContext.create(traceId, spanId, traceFlags, tracestateBuilder.build());
     } catch (IllegalArgumentException e) {

diff --git a/exporters/jaeger/build.gradle b/exporters/jaeger/build.gradle
@@ -54,7 +54,7 @@ protobuf {
 // generated Java directories as source folders.
 idea {
     module {
-        sourceDirs += file("build/generated/source/proto/main/java");
+        sourceDirs += file("build/generated/source/proto/main/java")
         // If you have additional sourceSets and/or codegen plugins, add all of them
     }
 }
diff --git a/proto/build.gradle b/proto/build.gradle
@@ -36,7 +36,7 @@ protobuf {
 // generated Java directories as source folders.
 idea {
     module {
-        sourceDirs += file("build/generated/source/proto/main/java");
+        sourceDirs += file("build/generated/source/proto/main/java")
         // If you have additional sourceSets and/or codegen plugins, add all of them
     }
 }
diff --git a/sdk/src/test/java/io/opentelemetry/sdk/trace/SpanBuilderSdkTest.java b/sdk/src/test/java/io/opentelemetry/sdk/trace/SpanBuilderSdkTest.java
@@ -44,7 +44,6 @@
 import org.junit.runners.JUnit4;
 
 /** Unit tests for {@link SpanBuilderSdk}. */
-@SuppressWarnings("deprecation")
 @RunWith(JUnit4.class)
 public class SpanBuilderSdkTest {
   private static final String SPAN_NAME = "span_name";