commit c1062a4d278517b26ff46e3bb866a0d05009b59c
author michajlo <michajlo@google.com> Wed Sep 26 14:53:38 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Wed Sep 26 14:55:23 2018 -0700
tree fc38a73aa0ed9f7f8f150875e494778d2bc3d2e7
parent 1ee4dd9c230be9add6a1e23cfc6662d58b231af6

Remove unused argument

PiperOrigin-RevId: 214674072

src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
index 65b7a9e..4f80a32 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/RuleContext.java
@@ -1493,7 +1493,7 @@
       target
           .getAssociatedRule()
           .getRuleClassObject()
-          .checkAttributesNonEmpty(target.getAssociatedRule(), reporter, attributes);
+          .checkAttributesNonEmpty(reporter, attributes);
     }
 
     public Builder setVisibility(NestedSet<PackageGroupContents> visibility) {

src/main/java/com/google/devtools/build/lib/packages/RuleClass.java

diff --git a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
index cb6b2b6..72d5401 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/RuleClass.java
@@ -1984,7 +1984,7 @@
   }
 
   public void checkAttributesNonEmpty(
-      Rule rule, RuleErrorConsumer ruleErrorConsumer, AttributeMap attributes) {
+      RuleErrorConsumer ruleErrorConsumer, AttributeMap attributes) {
     for (String attributeName : attributes.getAttributeNames()) {
       Attribute attr = attributes.getAttributeDefinition(attributeName);
       if (!attr.isNonEmpty()) {
@@ -1994,7 +1994,7 @@
 
       boolean isEmpty = false;
       if (attributeValue instanceof SkylarkList) {
-        isEmpty = ((SkylarkList) attributeValue).isEmpty();
+        isEmpty = ((SkylarkList<?>) attributeValue).isEmpty();
       } else if (attributeValue instanceof List<?>) {
         isEmpty = ((List<?>) attributeValue).isEmpty();
       } else if (attributeValue instanceof Map<?, ?>) {