(maven-enforcer) 01/01: Eliminate some string operations

[elharo]

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

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

commit 9601fb0a2ab1bf99d1e0e16cfa5a318acdaaf572
Author: Elliotte Rusty Harold <elh...@ibiblio.org>
AuthorDate: Sat Feb 22 19:39:29 2025 -0500

    Eliminate some string operations
---
 .../java/org/apache/maven/enforcer/rules/BannedRepositories.java | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git 
a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
 
b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
index 4b67f91..8347965 100644
--- 
a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
+++ 
b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
@@ -161,18 +161,17 @@ public final class BannedRepositories extends 
AbstractStandardEnforcerRule {
     }
 
     private String populateErrorMessage(List<ArtifactRepository> 
resultBannedRepos, String errorMessagePrefix) {
-        StringBuffer errMsg = new StringBuffer("");
         if (!resultBannedRepos.isEmpty()) {
-            errMsg.append("Current maven session contains banned" + 
errorMessagePrefix
+            return "Current maven session contains banned" + errorMessagePrefix
                     + "repository urls, please double check your pom or 
settings.xml:" + System.lineSeparator()
-                    + getRepositoryUrlString(resultBannedRepos) + 
System.lineSeparator() + System.lineSeparator());
+                    + getRepositoryUrlString(resultBannedRepos) + 
System.lineSeparator() + System.lineSeparator();
         }
 
-        return errMsg.toString();
+        return "";
     }
 
     private String getRepositoryUrlString(List<ArtifactRepository> 
resultBannedRepos) {
-        StringBuilder urls = new StringBuilder("");
+        StringBuilder urls = new StringBuilder();
         for (ArtifactRepository repo : resultBannedRepos) {
             urls.append(repo.getId() + " - " + repo.getUrl() + 
System.lineSeparator());
         }