aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build
diff options
context:
space:
mode:
authorGravatar vladmos <vladmos@google.com>2017-04-14 16:34:15 +0200
committerGravatar Klaus Aehlig <aehlig@google.com>2017-04-18 11:26:41 +0200
commit1341e3ed45045b85cda2c57c0623dae0d7149b16 (patch)
treeef7104e94efa4b18a4b665bb239f51a0babfb519 /src/main/java/com/google/devtools/build
parent1637af9b570197de5e8df888e4eb95fcf7c95234 (diff)
Fix `in` operator for targets
`DefaultInfo in target` should always return `true` for any target. PiperOrigin-RevId: 153166669
Diffstat (limited to 'src/main/java/com/google/devtools/build')
-rw-r--r--src/main/java/com/google/devtools/build/lib/analysis/AbstractConfiguredTarget.java10
1 files changed, 1 insertions, 9 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/AbstractConfiguredTarget.java b/src/main/java/com/google/devtools/build/lib/analysis/AbstractConfiguredTarget.java
index b522bce19d..acaa4a8db6 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/AbstractConfiguredTarget.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/AbstractConfiguredTarget.java
@@ -149,15 +149,7 @@ public abstract class AbstractConfiguredTarget
"Type Target only supports querying by object constructors, got %s instead",
EvalUtils.getDataTypeName(key)));
}
- ClassObjectConstructor constructor = (ClassObjectConstructor) key;
- SkylarkProviders provider = getProvider(SkylarkProviders.class);
- if (provider != null) {
- Object declaredProvider = provider.getDeclaredProvider(constructor.getKey());
- if (declaredProvider != null) {
- return true;
- }
- }
- return false;
+ return get(((ClassObjectConstructor) key).getKey()) != null;
}
@Override