diff options
author | 2017-01-13 12:22:10 +0000 | |
---|---|---|
committer | 2017-01-13 13:53:57 +0000 | |
commit | eb39663f30d4db7002ef757e1a4c66ca090ac19b (patch) | |
tree | fcd23ccb3524b8b843d7e7c9982d5176e119f2c9 /src/main/java | |
parent | bb1799333157345087249f82448db1982f92edb4 (diff) |
Remove a superfluous "return null" from JvmConfigurationLoader.
RedirectChaser needs to load the target to check if it's a bind/alias rule so it's not possible that it's not loaded yet by the time we get there.
--
PiperOrigin-RevId: 144429298
MOS_MIGRATED_REVID=144429298
Diffstat (limited to 'src/main/java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java b/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java index 6d9c42da5b..942d2abca3 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JvmConfigurationLoader.java @@ -90,9 +90,6 @@ public final class JvmConfigurationLoader implements ConfigurationFragmentFactor return null; } Target javaHomeTarget = lookup.getTarget(label); - if (javaHomeTarget == null) { - return null; - } if ((javaHomeTarget instanceof Rule) && "filegroup".equals(((Rule) javaHomeTarget).getRuleClass())) { RawAttributeMapper javaHomeAttributes = RawAttributeMapper.of((Rule) javaHomeTarget); |