commit 92a3a811330d296c4801c43627c72b76900c8181
author Alex Humesky <ahumesky@google.com> Mon May 09 19:07:56 2016 +0000
committer Klaus Aehlig <aehlig@google.com> Tue May 10 07:56:48 2016 +0000
tree d87fd624a383452970413089fc6df3ecaecb279d
parent 0595007b17931aff8c2412db21bb93f73c94c5b7

Fix OptionsParserImpl.clearValue (and invocation policy by extension) to work
correctly with flags that have expansion flags or implicit requirements.

--
MOS_MIGRATED_REVID=121862297

src/main/java/com/google/devtools/common/options/OptionsParser.java

diff --git a/src/main/java/com/google/devtools/common/options/OptionsParser.java b/src/main/java/com/google/devtools/common/options/OptionsParser.java
index fd6d915..83a7b08 100644
--- a/src/main/java/com/google/devtools/common/options/OptionsParser.java
+++ b/src/main/java/com/google/devtools/common/options/OptionsParser.java
@@ -593,7 +593,8 @@
    * @return A map of an option name to the old value of the options that were cleared.
    * @throws IllegalArgumentException If the flag does not exist.
    */
-  public Map<String, OptionValueDescription> clearValue(String optionName) {
+  public Map<String, OptionValueDescription> clearValue(String optionName)
+      throws OptionsParsingException {
     Map<String, OptionValueDescription> clearedValues = Maps.newHashMap();
     impl.clearValue(optionName, clearedValues);
     return clearedValues;