diff options
author | gjoranv <gv@verizonmedia.com> | 2019-08-22 12:26:51 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-08-22 12:26:51 +0200 |
commit | c0444cf8384a34981530d06743af26f163248744 (patch) | |
tree | 3fcdc5da100c6eb87ccb24139d81abd48e7e4cdc | |
parent | ddeeff77dffa68198e683e017e0b38ccce8b4bcc (diff) | |
parent | e5b271f06ff5422aa96f1690769ec6d91d85c2dc (diff) |
Merge pull request #10369 from vespa-engine/gjoranv/do-not-generate-findbugs-annotations
Don't add findbugs annotations to generated config classes.
3 files changed, 6 insertions, 8 deletions
diff --git a/configgen/src/main/java/com/yahoo/config/codegen/ConfigGenerator.java b/configgen/src/main/java/com/yahoo/config/codegen/ConfigGenerator.java index 4da546a7179..e6f305b3847 100644 --- a/configgen/src/main/java/com/yahoo/config/codegen/ConfigGenerator.java +++ b/configgen/src/main/java/com/yahoo/config/codegen/ConfigGenerator.java @@ -340,7 +340,7 @@ public class ConfigGenerator { "%s\n" +// "\n" +// " @Override\n" +// - " protected boolean doSetValue(@NonNull String name) {\n" +// + " protected boolean doSetValue(String name) {\n" +// " try {\n" +// " value = Enum.valueOf(name);\n" +// " return true;\n" +// diff --git a/configgen/src/main/java/com/yahoo/config/codegen/JavaClassBuilder.java b/configgen/src/main/java/com/yahoo/config/codegen/JavaClassBuilder.java index 40c903ee929..6df3ad5425a 100644 --- a/configgen/src/main/java/com/yahoo/config/codegen/JavaClassBuilder.java +++ b/configgen/src/main/java/com/yahoo/config/codegen/JavaClassBuilder.java @@ -72,7 +72,6 @@ public class JavaClassBuilder implements ClassBuilder { "import java.util.*;\n" + // "import java.io.File;\n" + // "import java.nio.file.Path;\n" + // - "import edu.umd.cs.findbugs.annotations.NonNull;\n" + // "import com.yahoo.config.*;"; } diff --git a/configgen/src/test/resources/allfeatures.reference b/configgen/src/test/resources/allfeatures.reference index f075daecd04..b84f01f380b 100644 --- a/configgen/src/test/resources/allfeatures.reference +++ b/configgen/src/test/resources/allfeatures.reference @@ -8,7 +8,6 @@ package com.yahoo.configgen; import java.util.*; import java.io.File; import java.nio.file.Path; -import edu.umd.cs.findbugs.annotations.NonNull; import com.yahoo.config.*; /** @@ -1112,7 +1111,7 @@ public final class AllfeaturesConfig extends ConfigInstance { public final static Enum FOOBAR = Enum.FOOBAR; @Override - protected boolean doSetValue(@NonNull String name) { + protected boolean doSetValue(String name) { try { value = Enum.valueOf(name); return true; @@ -1142,7 +1141,7 @@ public final class AllfeaturesConfig extends ConfigInstance { public final static Enum FOOBAR2 = Enum.FOOBAR2; @Override - protected boolean doSetValue(@NonNull String name) { + protected boolean doSetValue(String name) { try { value = Enum.valueOf(name); return true; @@ -1171,7 +1170,7 @@ public final class AllfeaturesConfig extends ConfigInstance { public final static Enum VALUES = Enum.VALUES; @Override - protected boolean doSetValue(@NonNull String name) { + protected boolean doSetValue(String name) { try { value = Enum.valueOf(name); return true; @@ -1731,7 +1730,7 @@ public final class AllfeaturesConfig extends ConfigInstance { public final static Enum TYPE = Enum.TYPE; @Override - protected boolean doSetValue(@NonNull String name) { + protected boolean doSetValue(String name) { try { value = Enum.valueOf(name); return true; @@ -2029,7 +2028,7 @@ public final class AllfeaturesConfig extends ConfigInstance { public final static Enum TYPE = Enum.TYPE; @Override - protected boolean doSetValue(@NonNull String name) { + protected boolean doSetValue(String name) { try { value = Enum.valueOf(name); return true; |