diff options
author | Alex Humesky <ahumesky@google.com> | 2015-11-11 20:26:45 +0000 |
---|---|---|
committer | Laszlo Csomor <laszlocsomor@google.com> | 2015-11-12 09:01:48 +0000 |
commit | fab4b888b4d0739e9269ecc0a6545e35fa185f66 (patch) | |
tree | 4bf7c302582999f49b314a2c35d26042df0634bc /src | |
parent | bbe821e6ac874e31e3d4d6473e89f1d3b7a49390 (diff) |
Deprecation warnings are no longer printed for flags specified using their old name.
--
MOS_MIGRATED_REVID=107611961
Diffstat (limited to 'src')
-rw-r--r-- | src/main/java/com/google/devtools/common/options/OptionsParserImpl.java | 10 | ||||
-rw-r--r-- | src/test/java/com/google/devtools/common/options/OptionsParserTest.java | 2 |
2 files changed, 0 insertions, 12 deletions
diff --git a/src/main/java/com/google/devtools/common/options/OptionsParserImpl.java b/src/main/java/com/google/devtools/common/options/OptionsParserImpl.java index e2cc6b8ce2..5f90594f1a 100644 --- a/src/main/java/com/google/devtools/common/options/OptionsParserImpl.java +++ b/src/main/java/com/google/devtools/common/options/OptionsParserImpl.java @@ -587,16 +587,6 @@ class OptionsParserImpl { value = "0"; } } - - // Add a deprecation warning if the old name for the flag is used. - if (field != null) { - Option option = field.getAnnotation(Option.class); - String oldName = option.oldName(); - if (name.equals(oldName)) { - addDeprecationWarning(oldName, "Use --" + option.name() + " instead"); - } - } - } else { throw new OptionsParsingException("Invalid options syntax: " + arg, arg); } diff --git a/src/test/java/com/google/devtools/common/options/OptionsParserTest.java b/src/test/java/com/google/devtools/common/options/OptionsParserTest.java index 396369467e..0787a398cc 100644 --- a/src/test/java/com/google/devtools/common/options/OptionsParserTest.java +++ b/src/test/java/com/google/devtools/common/options/OptionsParserTest.java @@ -1038,8 +1038,6 @@ public class OptionsParserTest { parser.parse("--old_name=foo"); OldNameExample result = parser.getOptions(OldNameExample.class); assertEquals("foo", result.flag); - assertThat(parser.getWarnings()).contains( - "Option 'old_name' is deprecated: Use --new_name instead"); // Should also work by its new name. parser = newOptionsParser(OldNameExample.class); |