diff options
Diffstat (limited to 'src/tools')
-rw-r--r-- | src/tools/skylark/java/com/google/devtools/skylark/skylint/UsageChecker.java | 3 | ||||
-rw-r--r-- | src/tools/skylark/javatests/com/google/devtools/skylark/skylint/UsageCheckerTest.java | 2 |
2 files changed, 4 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 a1e00adc27..0cc86b2043 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 @@ -189,7 +189,8 @@ public class UsageChecker extends AstVisitorWithNameResolution { if ("_".equals(name) || nameInfo.kind == Kind.BUILTIN) { return; } - if ((nameInfo.kind == Kind.LOCAL || nameInfo.kind == Kind.PARAMETER) && name.startsWith("_")) { + if ((nameInfo.kind == Kind.LOCAL || nameInfo.kind == Kind.PARAMETER) + && (name.startsWith("_") || 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 3fd9760d2c..ba7c87b845 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 @@ -286,6 +286,8 @@ public class UsageCheckerTest { @Test public void dontReportLocalsStartingWithUnderscore() throws Exception { 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(); } @Test |