Skip to content

Commit cff9b12

Browse files
authored
added support for '${name}' (#308)
1 parent 027311c commit cff9b12

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/checkstyle/checks/coding/UnusedLocalVarCheck.hx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,8 @@ class UnusedLocalVarCheck extends Check {
9090
return FOUND_GO_DEEPER;
9191
}
9292

93-
// '${name.doSomething()} or ${doSomething(name)}
94-
format = "\\$\\{(|.*[^_0-9a-zA-Z])" + name + "([^_0-9a-zA-Z].*)\\}";
93+
// '${name.doSomething()} or ${doSomething(name)} or ${name}
94+
format = "\\$\\{(|.*[^_0-9a-zA-Z])" + name + "([^_0-9a-zA-Z].*|)\\}";
9595
r = new EReg(format, "");
9696
if (r.match(s)) {
9797
return FOUND_GO_DEEPER;

test/checks/coding/UnusedLocalVarCheckTest.hx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,10 +65,12 @@ abstract UnusedLocalVarCheckTests(String) to String {
6565
var index:Int;
6666
var index2:Array<Int>;
6767
var index3:String;
68+
var index4:String;
6869
6970
trace ('$index');
7071
trace ('${index2.toString()}');
7172
trace ('${Std.parseInt(index3)}');
73+
trace ('${index4}');
7274
}
7375
}";
7476

@@ -81,6 +83,7 @@ abstract UnusedLocalVarCheckTests(String) to String {
8183
trace ('$index2');
8284
trace ('${index2.toString()}');
8385
trace ('${Std.parseInt(index3)}');
86+
trace ('${index4}');
8487
}
8588
}";
8689
}

0 commit comments

Comments
 (0)