fix: rename local variable to avoid shadowing sourcepath field

[sonarqube-agent]

This PR resolves a SonarQube MAJOR issue where a local variable named 'sourcepath' was shadowing the class field of the same name. The local variable at line 529 has been renamed to 'effectiveSourcepath' to eliminate the shadowing, improving code clarity and preventing potential confusion between the local variable and the class field.

View Project in SonarCloud

---

Fixed Issues

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

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

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 sourcepath at line 529 was shadowing the class field sourcepath declared at line 59. By renaming the local variable from sourcepath to effectiveSourcepath, the shadowing is eliminated, making the code clearer and avoiding potential confusion between the local variable and the class field.

--- a/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
+++ b/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
@@ -529,1 +529,1 @@ public class Enunciate implements Runnable {
-      List<File> sourcepath = this.sourcepath == null ? new ArrayList<>() : this.sourcepath;
+      List<File> effectiveSourcepath = this.sourcepath == null ? new ArrayList<>() : this.sourcepath;

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