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

gnodet pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven.git

commit 8f45d51bd60290cc3a95fa6dd530bf0362e86816
Author: Guillaume Nodet <[email protected]>
AuthorDate: Mon Jul 3 10:26:10 2023 +0200

    Prefer using String.isEmpty over String.length()>0
---
 api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java     | 2 +-
 .../src/main/java/org/apache/maven/internal/impl/DefaultArtifact.java   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java 
b/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java
index d645b89f0..73cf15d2e 100644
--- a/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java
+++ b/api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java
@@ -43,7 +43,7 @@ public interface Artifact {
                 + getArtifactId()
                 + ':'
                 + getExtension()
-                + (getClassifier().length() > 0 ? ":" + getClassifier() : "")
+                + (getClassifier().isEmpty() ? "" : ":" + getClassifier())
                 + ':'
                 + getVersion();
     }
diff --git 
a/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultArtifact.java 
b/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultArtifact.java
index 666e22722..ed865b81b 100644
--- 
a/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultArtifact.java
+++ 
b/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultArtifact.java
@@ -43,7 +43,7 @@ public class DefaultArtifact implements Artifact {
                 + getArtifactId()
                 + ':'
                 + getExtension()
-                + (getClassifier().length() > 0 ? ":" + getClassifier() : "")
+                + (getClassifier().isEmpty() ? "" : ":" + getClassifier())
                 + ':'
                 + getVersion();
     }

Reply via email to