fix: remove labeled break statement in Enunciate loop

[sonarqube-agent]

Removes the COPY_LOOP label and refactors the nested loop logic to eliminate unnecessary labeled breaks, improving code readability and maintainability. Using labels in Java is considered an anti-pattern that obscures control flow and makes the code harder to understand.

View Project in SonarCloud

---

Fixed Issues

java:S1119 - Refactor the code to remove this label and the need for it. • MAJOR • View issue

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

Why is this an issue?

Labels are not commonly used in Java, and many developers do not understand how they work. Moreover, their usage makes the control flow harder to follow, which reduces the code’s readability.

What changed

Removes the COPY_LOOP: label from the outer loop. This directly addresses the code smell about using labels in Java, which reduce readability and make control flow harder to follow.

--- a/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
+++ b/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
@@ -328,1 +327,0 @@ public class Enunciate implements Runnable {
-      COPY_LOOP:

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