(maven-checkstyle-plugin) branch master updated: Remove unused code (#158)

[elharo]

This is an automated email from the ASF dual-hosted git repository.

elharo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new d97687d  Remove unused code (#158)
d97687d is described below

commit d97687d5b96460cd23be444c40e488b716fef4b1
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Sun Nov 24 13:07:44 2024 +0000

    Remove unused code (#158)
    
    * Remove unused cod
---
 .../plugins/checkstyle/AbstractCheckstyleReport.java      | 15 ++++-----------
 .../plugins/checkstyle/CheckstyleViolationCheckMojo.java  |  3 +--
 2 files changed, 5 insertions(+), 13 deletions(-)

diff --git 
a/src/main/java/org/apache/maven/plugins/checkstyle/AbstractCheckstyleReport.java
 
b/src/main/java/org/apache/maven/plugins/checkstyle/AbstractCheckstyleReport.java
index 4ab9e05..d5f01ea 100644
--- 
a/src/main/java/org/apache/maven/plugins/checkstyle/AbstractCheckstyleReport.java
+++ 
b/src/main/java/org/apache/maven/plugins/checkstyle/AbstractCheckstyleReport.java
@@ -58,7 +58,6 @@ import org.codehaus.plexus.i18n.I18N;
 import org.codehaus.plexus.resource.ResourceManager;
 import org.codehaus.plexus.resource.loader.FileResourceLoader;
 import org.codehaus.plexus.util.FileUtils;
-import org.codehaus.plexus.util.PathTool;
 
 /**
  * Base abstract class for Checkstyle reports.
@@ -461,11 +460,13 @@ public abstract class AbstractCheckstyleReport extends 
AbstractMavenReport {
     protected ByteArrayOutputStream stringOutputStream;
 
     /** {@inheritDoc} */
+    @Override
     public String getName(Locale locale) {
         return getI18nString(locale, "name");
     }
 
     /** {@inheritDoc} */
+    @Override
     public String getDescription(Locale locale) {
         return getI18nString(locale, "description");
     }
@@ -479,6 +480,7 @@ public abstract class AbstractCheckstyleReport extends 
AbstractMavenReport {
         return i18n.getString("checkstyle-report", locale, 
"report.checkstyle." + key);
     }
 
+    @Override
     protected MavenProject getProject() {
         return project;
     }
@@ -488,6 +490,7 @@ public abstract class AbstractCheckstyleReport extends 
AbstractMavenReport {
     }
 
     /** {@inheritDoc} */
+    @Override
     public void executeReport(Locale locale) throws MavenReportException {
         checkDeprecatedParameterUsage(sourceDirectory, "sourceDirectory", 
"sourceDirectories");
         checkDeprecatedParameterUsage(testSourceDirectory, 
"testSourceDirectory", "testSourceDirectories");
@@ -675,16 +678,6 @@ public abstract class AbstractCheckstyleReport extends 
AbstractMavenReport {
         return consoleListener;
     }
 
-    private String determineRelativePath(File location) {
-        String relativePath =
-                
PathTool.getRelativePath(getReportOutputDirectory().getAbsolutePath(), 
location.getAbsolutePath());
-        if (relativePath == null || relativePath.trim().isEmpty()) {
-            relativePath = ".";
-        }
-
-        return relativePath + "/" + location.getName();
-    }
-
     protected List<File> getSourceDirectories() {
         if (sourceDirectories == null) {
             sourceDirectories = 
filterBuildTarget(project.getCompileSourceRoots());
diff --git 
a/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java
 
b/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java
index 21a9802..218fc27 100644
--- 
a/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java
+++ 
b/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java
@@ -485,11 +485,10 @@ public class CheckstyleViolationCheckMojo extends 
AbstractMojo {
     @Parameter(property = "checkstyle.excludeGeneratedSources", defaultValue = 
"false")
     private boolean excludeGeneratedSources;
 
-    private AuditListener auditListener;
-
     private File outputXmlFile;
 
     /** {@inheritDoc} */
+    @Override
     public void execute() throws MojoExecutionException, MojoFailureException {
         checkDeprecatedParameterUsage(sourceDirectory, "sourceDirectory", 
"sourceDirectories");
         checkDeprecatedParameterUsage(testSourceDirectory, 
"testSourceDirectory", "testSourceDirectories");