TouchInstinct
/
checkstyle
307
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Issue #2451: removed excess hierarchy from NoFinalizerCheck

This commit is contained in:
rnveach 2015-11-06 14:42:29 -05:00 committed by Roman Ivanov
parent be04ecad7f
commit 3fe171ec8a
1 changed files with 35 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/NoFinalizerCheck.java
View File

@ -19,6 +19,10 @@
package com.puppycrawl.tools.checkstyle.checks.coding;
import com.puppycrawl.tools.checkstyle.api.Check;
import com.puppycrawl.tools.checkstyle.api.DetailAST;
import com.puppycrawl.tools.checkstyle.api.TokenTypes;
/**
* Checks that no method having zero parameters is defined
* using the name <em>finalize</em>.
@ -27,7 +31,7 @@ package com.puppycrawl.tools.checkstyle.checks.coding;
* @author smckay@google.com (Steve McKay)
* @author lkuehne
*/
public class NoFinalizerCheck extends AbstractIllegalMethodCheck {
public class NoFinalizerCheck extends Check {
/**
* A key is pointing to the warning message text in "messages.properties"
@ -35,10 +39,35 @@ public class NoFinalizerCheck extends AbstractIllegalMethodCheck {
*/
public static final String MSG_KEY = "avoid.finalizer.method";
/**
* Creates an instance.
*/
public NoFinalizerCheck() {
super("finalize", MSG_KEY);
@Override
public int[] getDefaultTokens() {
return getAcceptableTokens();
}
@Override
public int[] getAcceptableTokens() {
return new int[] {TokenTypes.METHOD_DEF};
}
@Override
public int[] getRequiredTokens() {
return getAcceptableTokens();
}
@Override
public void visitToken(DetailAST aAST) {
final DetailAST mid = aAST.findFirstToken(TokenTypes.IDENT);
final String name = mid.getText();
if ("finalize".equals(name)) {
final DetailAST params = aAST.findFirstToken(TokenTypes.PARAMETERS);
final boolean hasEmptyParamList =
!params.branchContains(TokenTypes.PARAMETER_DEF);
if (hasEmptyParamList) {
log(aAST.getLineNo(), MSG_KEY);
}
}
}
}