fix: rename local variable to prevent shadowing of class field

[sonarqube-agent]

Resolved a SonarQube MAJOR issue where a local variable named 'sourceFiles' was shadowing a class field of the same name. Renamed the local variable to 'sourceFileURLs' to eliminate the shadowing and provide a more descriptive name that better reflects its Set type, improving code clarity and preventing potential bugs from accidental field reference confusion.

View Project in SonarCloud

---

Fixed Issues

java:S1117 - Rename "sourceFiles" which hides the field declared at line 54. • MAJOR • View issue

Location: core/src/main/java/com/webcohesion/enunciate/Enunciate.java:588

Why is this an issue?

Shadowing occurs when a local variable has the same name as a variable or a field in an outer scope.

What changed

This is the primary fix for the variable shadowing issue. The local variable sourceFiles at line 588 was shadowing the class field sourceFiles declared at line 54. By renaming the local variable from sourceFiles to sourceFileURLs, the shadowing is eliminated, giving the local variable a unique and more descriptive name that also better reflects its type (Set<URL>).

--- a/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
+++ b/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
@@ -588,1 +588,1 @@ public class Enunciate implements Runnable {
-      Set<URL> sourceFiles = getSourceFileURLs();
+      Set<URL> sourceFileURLs = getSourceFileURLs();

Have a suggestion or found an issue? Share your feedback here.

---

SonarQube Remediation Agent uses AI. Check for mistakes.

[sonarqubecloud]

SonarQube reviewer guide

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud