diff options
author | Ulf Adams <ulfjack@google.com> | 2015-08-21 11:03:37 +0000 |
---|---|---|
committer | Han-Wen Nienhuys <hanwen@google.com> | 2015-08-24 14:01:43 +0000 |
commit | c8a8c2a3be4aa473affd565311916953db27ec9b (patch) | |
tree | 03543b97fb0f16c6e2d5e4e1f97d8e622de70020 /src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java | |
parent | 6a4d36a4300a0b0e5c2acfece66b92bf0034f395 (diff) |
Prefer RuleContext.getFragment over BuildConfiguration.getFragment.
This improves the coverage of the legality check in RuleContext.getFragment.
--
MOS_MIGRATED_REVID=101208822
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java index 2f12d4a6fd..9ffaba58e7 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java @@ -80,10 +80,10 @@ public final class CppModel { private FeatureConfiguration featureConfiguration; public CppModel(RuleContext ruleContext, CppSemantics semantics) { - this.ruleContext = ruleContext; + this.ruleContext = Preconditions.checkNotNull(ruleContext); this.semantics = semantics; configuration = ruleContext.getConfiguration(); - cppConfiguration = configuration.getFragment(CppConfiguration.class); + cppConfiguration = ruleContext.getFragment(CppConfiguration.class); } /** |