A bit of clean-up

- Making empty final -- and static even though static is redundant here
- Removing unnecessary package qualification on forEach

Author: dougqh

internal-api/src/main/java/datadog/trace/api/TagMap.java

@@ -45,7 +45,7 @@
  */
 public interface TagMap extends Map<String, Object>, Iterable<TagMap.Entry> {
   /** Immutable empty TagMap - similar to {@link Collections#emptyMap()} */
-  TagMap EMPTY = TagMapFactory.INSTANCE.empty();
+  static final TagMap EMPTY = TagMapFactory.INSTANCE.empty();
 
   /** Creates a new mutable TagMap that contains the contents of <code>map</code> */
   static TagMap fromMap(Map<String, ?> map) {
@@ -2688,23 +2688,23 @@ public void fillStringMap(Map<? super String, ? super String> stringMap) {
   }
 
   @Override
-  public void forEach(Consumer<? super datadog.trace.api.TagMap.Entry> consumer) {
+  public void forEach(Consumer<? super TagMap.Entry> consumer) {
     for (Map.Entry<String, Object> entry : this.entrySet()) {
       consumer.accept(TagMap.Entry.newAnyEntry(entry));
     }
   }
 
   @Override
   public <T> void forEach(
-      T thisObj, BiConsumer<T, ? super datadog.trace.api.TagMap.Entry> consumer) {
+      T thisObj, BiConsumer<T, ? super TagMap.Entry> consumer) {
     for (Map.Entry<String, Object> entry : this.entrySet()) {
       consumer.accept(thisObj, TagMap.Entry.newAnyEntry(entry));
     }
   }
 
   @Override
   public <T, U> void forEach(
-      T thisObj, U otherObj, TriConsumer<T, U, ? super datadog.trace.api.TagMap.Entry> consumer) {
+      T thisObj, U otherObj, TriConsumer<T, U, ? super TagMap.Entry> consumer) {
     for (Map.Entry<String, Object> entry : this.entrySet()) {
       consumer.accept(thisObj, otherObj, TagMap.Entry.newAnyEntry(entry));
     }