diff --git a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/codestyle/UnnecessaryImportRule.java b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/codestyle/UnnecessaryImportRule.java index 8858928be3..cef8eea7c5 100644 --- a/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/codestyle/UnnecessaryImportRule.java +++ b/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/codestyle/UnnecessaryImportRule.java @@ -58,10 +58,8 @@ public class UnnecessaryImportRule extends AbstractJavaRule { private static final String IMPORT_FROM_SAME_PACKAGE_MESSAGE = "Unnecessary import from the current package ''{0}''"; private static final String IMPORT_FROM_JAVA_LANG_MESSAGE = "Unnecessary import from the java.lang package ''{0}''"; - private final Set staticImports = new HashSet<>(); private final Set allSingleNameImports = new HashSet<>(); private final Set allImportsOnDemand = new HashSet<>(); - private final Set staticImportsOnDemand = new HashSet<>(); private final Set unnecessaryJavaLangImports = new HashSet<>(); private final Set unnecessaryImportsFromSamePackage = new HashSet<>(); @@ -101,8 +99,6 @@ public class UnnecessaryImportRule extends AbstractJavaRule { @Override public Object visit(ASTCompilationUnit node, Object data) { this.allSingleNameImports.clear(); - this.staticImports.clear(); - this.staticImportsOnDemand.clear(); this.allImportsOnDemand.clear(); this.unnecessaryJavaLangImports.clear(); this.unnecessaryImportsFromSamePackage.clear(); @@ -214,12 +210,6 @@ public class UnnecessaryImportRule extends AbstractJavaRule { // duplicate reportWithMessage(node, data, DUPLICATE_IMPORT_MESSAGE); } - - if (node.isStatic()) { - container = node.isImportOnDemand() ? staticImportsOnDemand - : staticImports; - container.add(new ImportWrapper(node)); - } } private void reportWithMessage(ASTImportDeclaration node, Object data, String message) {