diff options
author | vladmos <vladmos@google.com> | 2017-04-14 16:34:15 +0200 |
---|---|---|
committer | Klaus Aehlig <aehlig@google.com> | 2017-04-18 11:26:41 +0200 |
commit | 1341e3ed45045b85cda2c57c0623dae0d7149b16 (patch) | |
tree | ef7104e94efa4b18a4b665bb239f51a0babfb519 /src/main/java/com/google/devtools/build | |
parent | 1637af9b570197de5e8df888e4eb95fcf7c95234 (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.java | 10 |
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 |