diff options
author | 2016-10-14 13:56:46 +0000 | |
---|---|---|
committer | 2016-10-14 20:24:51 +0000 | |
commit | 8d0be8960e49e09c0cbcc8173ce538edfa3cb208 (patch) | |
tree | 256bf07e82d4c2b93623ecd934d574291808cc46 /src | |
parent | 5e99198c66443e0298b406212c1e80a55fc729aa (diff) |
Rename field to clarify function.
--
MOS_MIGRATED_REVID=136152771
Diffstat (limited to 'src')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/ConfiguredRuleClassProvider.java | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/ConfiguredRuleClassProvider.java b/src/main/java/com/google/devtools/build/lib/analysis/ConfiguredRuleClassProvider.java index 2d3119c4ac..7f67e8dcc3 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/ConfiguredRuleClassProvider.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/ConfiguredRuleClassProvider.java @@ -56,7 +56,6 @@ import com.google.devtools.build.lib.syntax.Environment.Phase; import com.google.devtools.build.lib.syntax.Mutability; import com.google.devtools.build.lib.syntax.Type; import com.google.devtools.common.options.OptionsClassProvider; - import java.lang.reflect.Constructor; import java.lang.reflect.InvocationTargetException; import java.util.ArrayList; @@ -66,7 +65,6 @@ import java.util.List; import java.util.Map; import java.util.Set; import java.util.TreeMap; - import javax.annotation.Nullable; /** @@ -189,7 +187,8 @@ public class ConfiguredRuleClassProvider implements RuleClassProvider { private Label preludeLabel; private String runfilesPrefix; private String toolsRepository; - private final List<ConfigurationFragmentFactory> configurationFragments = new ArrayList<>(); + private final List<ConfigurationFragmentFactory> configurationFragmentFactories = + new ArrayList<>(); private final List<BuildInfoFactory> buildInfoFactories = new ArrayList<>(); private final List<Class<? extends FragmentOptions>> configurationOptions = new ArrayList<>(); @@ -278,7 +277,7 @@ public class ConfiguredRuleClassProvider implements RuleClassProvider { } public Builder addConfigurationFragment(ConfigurationFragmentFactory factory) { - configurationFragments.add(factory); + configurationFragmentFactories.add(factory); return this; } @@ -403,7 +402,7 @@ public class ConfiguredRuleClassProvider implements RuleClassProvider { defaultWorkspaceFileSuffix.toString(), ImmutableList.copyOf(buildInfoFactories), ImmutableList.copyOf(configurationOptions), - ImmutableList.copyOf(configurationFragments), + ImmutableList.copyOf(configurationFragmentFactories), configurationCollectionFactory, universalFragment, prerequisiteValidator, @@ -495,10 +494,8 @@ public class ConfiguredRuleClassProvider implements RuleClassProvider { */ private final ImmutableList<Class<? extends FragmentOptions>> configurationOptions; - /** - * The set of configuration fragment factories. - */ - private final ImmutableList<ConfigurationFragmentFactory> configurationFragments; + /** The set of configuration fragment factories. */ + private final ImmutableList<ConfigurationFragmentFactory> configurationFragmentFactories; /** * The factory that creates the configuration collection. @@ -548,7 +545,7 @@ public class ConfiguredRuleClassProvider implements RuleClassProvider { this.defaultWorkspaceFileSuffix = defaultWorkspaceFileSuffix; this.buildInfoFactories = buildInfoFactories; this.configurationOptions = configurationOptions; - this.configurationFragments = configurationFragments; + this.configurationFragmentFactories = configurationFragments; this.configurationCollectionFactory = configurationCollectionFactory; this.universalFragment = universalFragment; this.prerequisiteValidator = prerequisiteValidator; @@ -601,7 +598,7 @@ public class ConfiguredRuleClassProvider implements RuleClassProvider { * Returns the set of configuration fragments provided by this module. */ public ImmutableList<ConfigurationFragmentFactory> getConfigurationFragments() { - return configurationFragments; + return configurationFragmentFactories; } /** |