You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2021/10/13 20:54:00 UTC

[maven] branch master updated: [MNG-7298] - Avoid redundant conditions.

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 4c21e11  [MNG-7298] - Avoid redundant conditions.
4c21e11 is described below

commit 4c21e11b011999d8455d4406a07bcc2e6eaa73ae
Author: Arturo Bernal <ar...@gmail.com>
AuthorDate: Mon Oct 11 21:30:26 2021 +0200

    [MNG-7298] - Avoid redundant conditions.
---
 .../main/java/org/apache/maven/artifact/versioning/VersionRange.java    | 2 +-
 .../repository/legacy/resolver/DefaultLegacyArtifactCollector.java      | 2 +-
 .../java/org/apache/maven/repository/metadata/MetadataGraphVertex.java  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java b/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java
index 3bf0658..67bde82 100644
--- a/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java
+++ b/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java
@@ -160,7 +160,7 @@ public class VersionRange
 
             process = process.substring( index + 1 ).trim();
 
-            if ( process.length() > 0 && process.startsWith( "," ) )
+            if ( process.startsWith( "," ) )
             {
                 process = process.substring( 1 ).trim();
             }
diff --git a/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java b/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
index 69c72bf..048ce8f 100644
--- a/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
+++ b/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java
@@ -214,7 +214,7 @@ public class DefaultLegacyArtifactCollector
                                                      Map<String, Artifact> managedVersions )
     {
         ManagedVersionMap versionMap;
-        if ( ( managedVersions != null ) && ( managedVersions instanceof ManagedVersionMap ) )
+        if ( managedVersions instanceof ManagedVersionMap )
         {
             versionMap = (ManagedVersionMap) managedVersions;
         }
diff --git a/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataGraphVertex.java b/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataGraphVertex.java
index 495d061..f24c395 100644
--- a/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataGraphVertex.java
+++ b/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataGraphVertex.java
@@ -175,7 +175,7 @@ public class MetadataGraphVertex
     @Override
     public boolean equals( Object vo )
     {
-        if ( vo == null || !( vo instanceof MetadataGraphVertex ) )
+        if ( !( vo instanceof MetadataGraphVertex ) )
         {
             return false;
         }