diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/main/java/com/google/devtools/common/options/OptionsParserImpl.java | 14 |
1 files changed, 8 insertions, 6 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 d3a3751b16..a7745f6fe6 100644 --- a/src/main/java/com/google/devtools/common/options/OptionsParserImpl.java +++ b/src/main/java/com/google/devtools/common/options/OptionsParserImpl.java @@ -341,8 +341,15 @@ class OptionsParserImpl { OptionDefinition optionDefinition = parsedOption.getOptionDefinition(); // All options can be deprecated; check and warn before doing any option-type specific work. maybeAddDeprecationWarning(optionDefinition); - @Nullable String unconvertedValue = parsedOption.getUnconvertedValue(); + // Track the value, before any remaining option-type specific work that is done outside of + // the OptionValueDescription. + OptionValueDescription entry = + optionValues.computeIfAbsent( + optionDefinition, OptionValueDescription::createOptionValueDescription); + entry.addOptionInstance(parsedOption, implicitDependent, expandedFrom, warnings); + + @Nullable String unconvertedValue = parsedOption.getUnconvertedValue(); if (optionDefinition.isWrapperOption()) { if (unconvertedValue.startsWith("-")) { String sourceMessage = @@ -414,11 +421,6 @@ class OptionsParserImpl { + ": " + Joiner.on(' ').join(unparsed)); } - } else { - OptionValueDescription entry = - optionValues.computeIfAbsent( - optionDefinition, OptionValueDescription::createOptionValueDescription); - entry.addOptionInstance(parsedOption, implicitDependent, expandedFrom, warnings); } // Collect any implicit requirements. |