Merge pull request #722 from hazendaz/master

[cleanup] Cleanup location of properties or remove unused
spotbugs · Dec 25, 2023 · b5b0fd1 · b5b0fd1
2 parents 0e6f4d6 + 4e99998
commit b5b0fd1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/src/main/groovy/org/codehaus/mojo/spotbugs/SpotBugsPluginsTrait.groovy b/src/main/groovy/org/codehaus/mojo/spotbugs/SpotBugsPluginsTrait.groovy
@@ -57,8 +57,6 @@ trait SpotBugsPluginsTrait {
     String getSpotbugsPlugins() {
         ResourceHelper resourceHelper = new ResourceHelper(log, spotbugsXmlOutputDirectory, resourceManager)
 
-        URL[] pluginURL
-
         String urlPlugins = ""
 
         if (pluginList) {

diff --git a/src/main/groovy/org/codehaus/mojo/spotbugs/SpotbugsReportGenerator.groovy b/src/main/groovy/org/codehaus/mojo/spotbugs/SpotbugsReportGenerator.groovy
@@ -438,15 +438,13 @@ class SpotbugsReportGenerator implements SpotBugsInfo {
      *
      */
     protected String assembleJxrHyperlink(GPathResult line) {
-        String hyperlink
-        String prefix
-
         log.debug("Inside assembleJxrHyperlink")
         log.debug("line is " + line.text())
         log.debug("outputDirectory is " + outputDirectory.getAbsolutePath())
         log.debug("xrefLocation is " + xrefLocation.getAbsolutePath())
         log.debug("xrefTestLocation is " + xrefTestLocation.getAbsolutePath())
 
+        String prefix
         compileSourceRoots.each { compileSourceRoot ->
             if (!new File(compileSourceRoot + File.separator + line.@sourcepath.text()).exists()) {
                 return
@@ -468,6 +466,7 @@ class SpotbugsReportGenerator implements SpotBugsInfo {
         String path = prefix + line.@classname.text().replaceAll("[.]", "/").replaceAll("[\$].*", "")
         String lineNumber = valueForLine(line)
 
+        String hyperlink
         if (lineNumber != bundle.getString(NOLINE_KEY)) {
             hyperlink = "<a href=\"" + path + ".html#L" + line.@start.text() + "\">" + lineNumber + "</a>"
         } else {