diff options
author | Marcel Hlopko <hlopko@google.com> | 2016-11-16 15:48:25 +0000 |
---|---|---|
committer | Kristina Chodorow <kchodorow@google.com> | 2016-11-16 16:07:04 +0000 |
commit | b782fd45f7046a94a617ec8aec76ee5513555924 (patch) | |
tree | df8cc2fe27d862d35302a8dd116d1dfd47f4ef82 /src/main/protobuf/test_status.proto | |
parent | 4e8e67bcf25ea4e06c16104fc7f4db528a16508f (diff) |
Simplify Crosstool Build Variables context
This cl merges two classes used to hold build variables context into one. Those
classes are (previously top-level) jcgd.build.lib.rules.cpp.Variables, and its
inner class View. Both represent a collection of build variables and imo it
makes sense to merge them to make the code simpler.
Also, I cleaned up the build variables api to use primitive strings or instances of VariableValueBuilders, and I've hidden subclasses of VariableValues.
Last but not least, I refactored the code to use immutable collections exclusively. That revealed that 'module_files' variable is sometimes registered twice. I want to clean this eventually ([]).
--
MOS_MIGRATED_REVID=139329823
Diffstat (limited to 'src/main/protobuf/test_status.proto')
0 files changed, 0 insertions, 0 deletions