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 2020/01/26 01:29:11 UTC
[maven] 02/02: Remove redundant code
This is an automated email from the ASF dual-hosted git repository.
slachiewicz pushed a commit to branch simplify_code
in repository https://gitbox.apache.org/repos/asf/maven.git
commit 427e10f9fb7645cb983968560a5d41bd9a2e8602
Author: thelproad <th...@outlook.com>
AuthorDate: Sat Aug 17 17:04:15 2019 +0800
Remove redundant code
Closes #280
---
.../artifact/repository/metadata/AbstractRepositoryMetadata.java | 3 +--
.../java/org/apache/maven/classrealm/DefaultClassRealmManager.java | 5 ++---
.../org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java | 2 +-
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java | 4 ++--
4 files changed, 6 insertions(+), 8 deletions(-)
diff --git a/maven-core/src/main/java/org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java b/maven-core/src/main/java/org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java
index 46c41b9..196b4b7 100644
--- a/maven-core/src/main/java/org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java
+++ b/maven-core/src/main/java/org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java
@@ -123,8 +123,7 @@ public abstract class AbstractRepositoryMetadata
// beware meta-versions!
String version = metadata.getVersion();
- if ( version != null && ( Artifact.LATEST_VERSION.equals( version ) || Artifact.RELEASE_VERSION.equals(
- version ) ) )
+ if ( Artifact.LATEST_VERSION.equals( version ) || Artifact.RELEASE_VERSION.equals( version ) )
{
// meta-versions are not valid <version/> values...don't write them.
metadata.setVersion( null );
diff --git a/maven-core/src/main/java/org/apache/maven/classrealm/DefaultClassRealmManager.java b/maven-core/src/main/java/org/apache/maven/classrealm/DefaultClassRealmManager.java
index a4835fd..cf3d746 100644
--- a/maven-core/src/main/java/org/apache/maven/classrealm/DefaultClassRealmManager.java
+++ b/maven-core/src/main/java/org/apache/maven/classrealm/DefaultClassRealmManager.java
@@ -243,12 +243,11 @@ public class DefaultClassRealmManager
{
Objects.requireNonNull( plugin, "plugin cannot be null" );
- ClassLoader parent = PARENT_CLASSLOADER;
-
Map<String, ClassLoader> foreignImports =
Collections.<String, ClassLoader>singletonMap( "", getMavenApiRealm() );
- return createRealm( getKey( plugin, true ), RealmType.Extension, parent, null, foreignImports, artifacts );
+ return createRealm( getKey( plugin, true ), RealmType.Extension, PARENT_CLASSLOADER, null,
+ foreignImports, artifacts );
}
private boolean isProvidedArtifact( Artifact artifact )
diff --git a/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java b/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java
index edb8dce..4bafa30 100644
--- a/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java
+++ b/maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoDescriptorCreator.java
@@ -190,7 +190,7 @@ public class MojoDescriptorCreator
plugin.setArtifactId( tok.nextToken() );
goal = tok.nextToken();
}
- else if ( numTokens <= 2 )
+ else
{
// We have a prefix and goal
//
diff --git a/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java b/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java
index 5ca0602..79e1ffe 100644
--- a/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java
+++ b/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java
@@ -337,7 +337,7 @@ public class MavenCli
"-D%s system property is not set.", MULTIMODULE_PROJECT_DIRECTORY );
throw new ExitException( 1 );
}
- File basedir = basedirProperty != null ? new File( basedirProperty ) : new File( "" );
+ File basedir = new File( basedirProperty );
try
{
cliRequest.multiModuleProjectDirectory = basedir.getCanonicalFile();
@@ -764,7 +764,7 @@ public class MavenCli
MavenExecutionRequest request = DefaultMavenExecutionRequest.copy( cliRequest.request );
- request = populateRequest( cliRequest, request );
+ populateRequest( cliRequest, request );
request = executionRequestPopulator.populateDefaults( request );