diff options
| author |  jmmv <jmmv@google.com> | 2018-04-19 12:32:59 -0700 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-04-19 12:34:13 -0700 |
| commit | 64553797c63845d00c8ed162e4ddfae03295e8b9 (patch) | |
| tree | 63f9bbd2a79e1fbce45858d58cd24d22f37781e6 /src/main/java/com/google/devtools/build/lib/query2/ConfiguredTargetQueryEnvironment.java | |
| parent | 788b63a8c90e57dba511de82afd47ea73680c906 (diff) | |
Reduce the state kept in OptionProcessor.
Make the list of rc files a local variable as it need not be a class
attribute, and drop the unused rcoptions_ field.
This is a trivial refactoring and the remaining code is still too
confusing. It'd be worth splitting OptionProcessor in two pieces:
OptionProcessor to exclusively keep the virtual ParseOptions method
and no state, and a new ParsedOptions type to act as the immutable
return value of ParseOptions. This would decouple all state
mutations.
RELNOTES: None.
PiperOrigin-RevId: 193557347
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/query2/ConfiguredTargetQueryEnvironment.java')
0 files changed, 0 insertions, 0 deletions