diff options
author | 2016-04-12 08:34:23 +0000 | |
---|---|---|
committer | 2016-04-12 13:59:04 +0000 | |
commit | 6f26035ca8c8940eb6c9ed1c4549437d65c0a2d4 (patch) | |
tree | de19d4299cca9bc866698ff5543a69567d2808bb /src/main/java/com | |
parent | d9a644fef5bb08087c2470e8f904c34dffb0f6da (diff) |
Remove some dead code.
The cast to Label can never succeed for a configured target node, so this
code could never have been executed (or we'd see crashes at this location).
--
MOS_MIGRATED_REVID=119615022
Diffstat (limited to 'src/main/java/com')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java index 894a653acc..fc00b5760c 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java @@ -391,17 +391,7 @@ public final class SkyframeBuildView { if (culprit.functionName().equals(SkyFunctions.CONFIGURED_TARGET)) { return ((ConfiguredTargetKey) culprit.argument()).getLabel(); } else { - // For other types of cycles (e.g. file symlink cycles), the root cause is the furthest - // target dependency that itself depended on the cycle. - Label furthestTarget = labelToLoad; - for (SkyKey skyKey : cycleInfo.getPathToCycle()) { - if (skyKey.functionName().equals(SkyFunctions.CONFIGURED_TARGET)) { - furthestTarget = (Label) skyKey.argument(); - } else { - break; - } - } - return furthestTarget; + return labelToLoad; } } return null; |