diff options
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java b/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java index 20b9304fc9..f03ee6bf08 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java +++ b/src/main/java/com/google/devtools/build/lib/packages/AbstractAttributeMapper.java @@ -134,10 +134,10 @@ public abstract class AbstractAttributeMapper implements AttributeMap { for (Attribute attribute : ruleClass.getAttributes()) { Type<?> type = attribute.getType(); // TODO(bazel-team): This is incoherent: we shouldn't have to special-case these types - // for our visitation policy (e.g., why is Type.NODEP_LABEL_LIST excluded but not - // Type.NODEP_LABEL?). But this is the semantics the calling code requires. Audit + // for our visitation policy. But this is the semantics the calling code requires. Audit // exactly which calling code expects what and clean up this interface. - if (type == Type.OUTPUT || type == Type.OUTPUT_LIST || type == Type.NODEP_LABEL_LIST) { + if (type == Type.OUTPUT || type == Type.OUTPUT_LIST + || type == Type.NODEP_LABEL || type == Type.NODEP_LABEL_LIST) { continue; } for (Object value : visitAttribute(attribute.getName(), type)) { |