Remove unused imports and variables

cflint · Mar 25, 2024 · fff884d · fff884d
1 parent 63a8802
commit fff884d
Show file tree

Hide file tree

Showing 6 changed files with 418 additions and 426 deletions.
diff --git a/src/main/java/com/cflint/cli/CFLintCLI.java b/src/main/java/com/cflint/cli/CFLintCLI.java
@@ -21,12 +21,10 @@
 import com.cflint.Version;
 import com.cflint.api.CFLintAPI;
 import com.cflint.api.CFLintResult;
-import com.cflint.cli.Settings;
 import com.cflint.config.CFLintConfiguration;
 import com.cflint.config.CFLintPluginInfo;
 import com.cflint.config.CFLintPluginInfo.PluginInfoRule;
 import com.cflint.config.CFLintPluginInfo.PluginInfoRule.PluginMessage;
-import com.cflint.config.CFLintPluginInfo.RuleGroup;
 import com.cflint.config.ConfigBuilder;
 import com.cflint.config.ConfigUtils;
 import com.cflint.exception.CFLintConfigurationException;

diff --git a/src/main/java/com/cflint/plugins/core/ArgVarChecker.java b/src/main/java/com/cflint/plugins/core/ArgVarChecker.java
@@ -7,11 +7,9 @@
 import com.cflint.plugins.CFLintScannerAdapter;
 import com.cflint.plugins.Context;
 
-import cfml.parsing.cfscript.CFAssignmentExpression;
 import cfml.parsing.cfscript.CFExpression;
 import cfml.parsing.cfscript.CFFullVarExpression;
 import cfml.parsing.cfscript.CFIdentifier;
-import cfml.parsing.cfscript.CFVarDeclExpression;
 
 public class ArgVarChecker extends CFLintScannerAdapter {
 

diff --git a/src/main/java/com/cflint/plugins/core/FunctionCollisionChecker.java b/src/main/java/com/cflint/plugins/core/FunctionCollisionChecker.java
@@ -10,8 +10,6 @@
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
 
-import cfml.parsing.cfscript.CFExpression;
-import cfml.parsing.cfscript.CFFunctionExpression;
 import cfml.parsing.cfscript.script.CFFuncDeclStatement;
 import cfml.parsing.cfscript.script.CFScriptStatement;
 import net.htmlparser.jericho.Element;

diff --git a/src/main/java/com/cflint/plugins/core/UnusedArgumentChecker.java b/src/main/java/com/cflint/plugins/core/UnusedArgumentChecker.java
@@ -1,6 +1,5 @@
 package com.cflint.plugins.core;
 
-import java.util.HashMap;
 import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.Map.Entry;
@@ -34,7 +33,7 @@ static class ArgInfo{
         String type;
         String casedName;
     }
-    
+
     @Override
     public void element(final Element element, final Context context, final BugList bugs) {
         if (element.getName().equals(CF.CFARGUMENT)) {
@@ -43,7 +42,7 @@ public void element(final Element element, final Context context, final BugList
             ArgInfo argInfo = new ArgInfo();
             argInfo.casedName=name;
             argInfo.argumentLineNo=context.startLine();
-            argInfo.argumentOffset=element.getAttributeValue(CF.NAME) != null 
+            argInfo.argumentOffset=element.getAttributeValue(CF.NAME) != null
                     ? element.getAttributes().get(CF.NAME).getValueSegment().getBegin() : element.getBegin();
             argInfo.type=element.getAttributeValue(CF.TYPE);
             currentArgs.put(name.toLowerCase(), argInfo);
@@ -59,7 +58,7 @@ public void expression(final CFScriptStatement expression, final Context context
         if (expression instanceof CFFuncDeclStatement) {
             final CFFuncDeclStatement function = (CFFuncDeclStatement) expression;
             for (final CFFunctionParameter argument : function.getFormals()) {
-                final String name = argument.getName().toLowerCase(); 
+                final String name = argument.getName().toLowerCase();
                 // CF variable names are not case sensitive
                 ArgInfo argInfo = new ArgInfo();
                 argInfo.casedName=argument.getName();
@@ -97,7 +96,7 @@ protected void useIdentifier(final CFIdentifier identifier) {
             currentArgs.get(name).used=true;
         }
     }
-    
+
     protected void useIdentifier(final CFFunctionExpression identifier) {
         final String name = identifier.getName().toLowerCase();
         if (currentArgs.get(name) != null && CF.FUNCTION.equalsIgnoreCase(currentArgs.get(name).type)) {