Merge pull request #812 from sebright/remove-tagkey-todo

sebright2 · web-flow · commit 4c36f49f34fe · 2017-11-17T13:21:26.000-08:00
Remove an obsolete TODO from TagKey.java.
diff --git a/api/src/main/java/io/opencensus/tags/TagKey.java b/api/src/main/java/io/opencensus/tags/TagKey.java
@@ -54,7 +54,6 @@ public abstract class TagKey {
    * @throws IllegalArgumentException if the name is not valid.
    */
   public static TagKey create(String name) {
-    // TODO(sebright): Should we disallow an empty name?
     checkArgument(isValid(name));
     return new AutoValue_TagKey(name);
   }

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,6 @@ public abstract class TagKey {`
`54`	`54`	`* @throws IllegalArgumentException if the name is not valid.`
`55`	`55`	`*/`
`56`	`56`	`public static TagKey create(String name) {`
`57`		`- // TODO(sebright): Should we disallow an empty name?`
`58`	`57`	`checkArgument(isValid(name));`
`59`	`58`	`return new AutoValue_TagKey(name);`
`60`	`59`	`}`