diff options
| -rw-r--r-- | src/tools/skylark/java/com/google/devtools/skylark/skylint/UsageChecker.java | 2 | ||||
| -rw-r--r-- | src/tools/skylark/javatests/com/google/devtools/skylark/skylint/UsageCheckerTest.java | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/src/tools/skylark/java/com/google/devtools/skylark/skylint/UsageChecker.java b/src/tools/skylark/java/com/google/devtools/skylark/skylint/UsageChecker.java index d8fbfede69..0c62e73bcd 100644 --- a/src/tools/skylark/java/com/google/devtools/skylark/skylint/UsageChecker.java +++ b/src/tools/skylark/java/com/google/devtools/skylark/skylint/UsageChecker.java @@ -190,7 +190,7 @@ public class UsageChecker extends AstVisitorWithNameResolution { return; } if ((nameInfo.kind == Kind.LOCAL || nameInfo.kind == Kind.PARAMETER) - && (name.startsWith("_") || name.startsWith("unused_"))) { + && (name.startsWith("_") || name.startsWith("unused_") || name.startsWith("UNUSED_"))) { // local variables starting with an underscore need not be used return; } diff --git a/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/UsageCheckerTest.java b/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/UsageCheckerTest.java index 0a74befd40..3acf1d93d7 100644 --- a/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/UsageCheckerTest.java +++ b/src/tools/skylark/javatests/com/google/devtools/skylark/skylint/UsageCheckerTest.java @@ -306,6 +306,8 @@ public class UsageCheckerTest { Truth.assertThat(findIssues("def f(_param):", " _local = [[] for _x in []]")).isEmpty(); Truth.assertThat(findIssues("def f(unused_param):", " unused_local = [[] for unused_x in []]")) .isEmpty(); + Truth.assertThat(findIssues("def f():", " UNUSED_CONSTANT = 'unused'")) + .isEmpty(); } @Test |