diff options
| author |  Greg Estren <gregce@google.com> | 2016-05-20 19:27:46 +0000 |
|---|---|---|
| committer |  Yue Gan <yueg@google.com> | 2016-05-23 08:24:40 +0000 |
| commit | f2652b2048947eb614829876051b27f1a753bd06 (patch) | |
| tree | 48c69163930d67b19ee806c78e98b070d5da47dc /src/main/java/com/google/devtools/build/lib | |
| parent | 243c6f2f906f8c9f4300a6b833def791d117b321 (diff) | |
When BuildOptions.get(Class<T> optionsClass) fails, include which
fragment was requested in the NullPointerException message.
This helps diagnose dynamic configuration errors.
--
MOS_MIGRATED_REVID=122855814
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java index 9272188cd7..46842ae6bd 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java @@ -153,7 +153,7 @@ public final class BuildOptions implements Cloneable, Serializable { */ public <T extends FragmentOptions> T get(Class<T> optionsClass) { FragmentOptions options = fragmentOptionsMap.get(optionsClass); - Preconditions.checkNotNull(options); + Preconditions.checkNotNull(options, "fragment options unavailable: " + optionsClass.getName()); return optionsClass.cast(options); } |