Remove type checking requirement from AttributeMap.has.

This overrides the traditional has(String name, Type<>T> type)
with has(String name) and removes the type check outright from
isConfigurable.

Ideally we'd remove the old version in this same change. But there
are enough uses of it that that's not a risk-free change and
is safer as followup changes.

--
PiperOrigin-RevId: 147513593
MOS_MIGRATED_REVID=147513593

1 files changed, 8 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/packages/DelegatingAttributeMapper.java b/src/main/java/com/google/devtools/build/lib/packages/DelegatingAttributeMapper.java
index e57476c551..3f0d2b0eef 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/DelegatingAttributeMapper.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/DelegatingAttributeMapper.java
@@ -46,8 +46,8 @@ public class DelegatingAttributeMapper implements AttributeMap {
   }
 
   @Override
-  public <T> boolean isConfigurable(String attributeName, Type<T> type) {
-    return delegate.isConfigurable(attributeName, type);
+  public boolean isConfigurable(String attributeName) {
+    return delegate.isConfigurable(attributeName);
   }
 
   @Override
@@ -98,7 +98,12 @@ public class DelegatingAttributeMapper implements AttributeMap {
   }
 
   @Override
-  public boolean has(String attrName, Type<?> type) {
+  public boolean has(String attrName) {
+    return delegate.has(attrName);
+  }
+
+  @Override
+  public <T> boolean has(String attrName, Type<T> type) {
     return delegate.has(attrName, type);
   }
 }