diff options
author | 2016-08-17 17:04:27 +0000 | |
---|---|---|
committer | 2016-08-18 08:29:43 +0000 | |
commit | 88f4357431240c9339284c158776677d34061cbf (patch) | |
tree | bfd0ddf2bb918ccbe3cf35ad36b57bf45bc6c73f /src/main/java/com/google/devtools/build | |
parent | 4e42cdd93d7f62b13971d0ba1a5e1febc582dc61 (diff) |
Check that dependencies are non-null upon insertion.
Otherwise, we'll get an NPE in build(), which doesn't help in finding the place where the null was added.
--
MOS_MIGRATED_REVID=130531765
Diffstat (limited to 'src/main/java/com/google/devtools/build')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java index d337bbeaa6..c19fac1ae3 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaLibraryHelper.java @@ -14,6 +14,7 @@ package com.google.devtools.build.lib.rules.java; +import static com.google.common.base.Preconditions.checkNotNull; import static com.google.devtools.build.lib.analysis.config.BuildConfiguration.StrictDepsMode.OFF; import static com.google.devtools.build.lib.rules.java.JavaCompilationArgs.ClasspathType.BOTH; @@ -92,6 +93,7 @@ public final class JavaLibraryHelper { } public JavaLibraryHelper addDep(JavaCompilationArgsProvider provider) { + checkNotNull(provider); this.deps.add(provider); return this; } |