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-resolver.git
The following commit(s) were added to refs/heads/master by this push:
new 253871b Fix bad requireNonNull checks
253871b is described below
commit 253871b5a1184f39e0a95428d008647bb0767419
Author: Guillaume Nodet <[email protected]>
AuthorDate: Wed Feb 16 06:13:29 2022 +0100
Fix bad requireNonNull checks
---
.../aether/internal/impl/EnhancedLocalRepositoryManagerFactory.java | 4 ++--
.../aether/internal/impl/SimpleLocalRepositoryManagerFactory.java | 4 ++--
.../eclipse/aether/spi/connector/transport/AbstractTransporter.java | 6 +++---
.../aether/transport/classpath/ClasspathTransporterFactory.java | 4 ++--
.../org/eclipse/aether/transport/file/FileTransporterFactory.java | 4 ++--
.../java/org/eclipse/aether/transport/wagon/WagonTransporter.java | 2 +-
.../org/eclipse/aether/transport/wagon/WagonTransporterFactory.java | 4 ++--
7 files changed, 14 insertions(+), 14 deletions(-)
diff --git
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/EnhancedLocalRepositoryManagerFactory.java
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/EnhancedLocalRepositoryManagerFactory.java
index 56ace05..d5d64b1 100644
---
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/EnhancedLocalRepositoryManagerFactory.java
+++
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/EnhancedLocalRepositoryManagerFactory.java
@@ -69,8 +69,8 @@ public class EnhancedLocalRepositoryManagerFactory
public LocalRepositoryManager newInstance( RepositorySystemSession
session, LocalRepository repository )
throws NoLocalRepositoryManagerException
{
- Objects.requireNonNull( "session", "session cannot be null" );
- Objects.requireNonNull( "repository", "repository cannot be null" );
+ Objects.requireNonNull( session, "session cannot be null" );
+ Objects.requireNonNull( repository, "repository cannot be null" );
if ( "".equals( repository.getContentType() ) || "default".equals(
repository.getContentType() ) )
{
diff --git
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/SimpleLocalRepositoryManagerFactory.java
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/SimpleLocalRepositoryManagerFactory.java
index 1be1bce..6903e4c 100644
---
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/SimpleLocalRepositoryManagerFactory.java
+++
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/SimpleLocalRepositoryManagerFactory.java
@@ -48,8 +48,8 @@ public class SimpleLocalRepositoryManagerFactory
public LocalRepositoryManager newInstance( RepositorySystemSession
session, LocalRepository repository )
throws NoLocalRepositoryManagerException
{
- Objects.requireNonNull( "session", "session cannot be null" );
- Objects.requireNonNull( "repository", "repository cannot be null" );
+ Objects.requireNonNull( session, "session cannot be null" );
+ Objects.requireNonNull( repository, "repository cannot be null" );
if ( "".equals( repository.getContentType() ) || "simple".equals(
repository.getContentType() ) )
{
diff --git
a/maven-resolver-spi/src/main/java/org/eclipse/aether/spi/connector/transport/AbstractTransporter.java
b/maven-resolver-spi/src/main/java/org/eclipse/aether/spi/connector/transport/AbstractTransporter.java
index 45389a6..4bb276c 100644
---
a/maven-resolver-spi/src/main/java/org/eclipse/aether/spi/connector/transport/AbstractTransporter.java
+++
b/maven-resolver-spi/src/main/java/org/eclipse/aether/spi/connector/transport/AbstractTransporter.java
@@ -48,7 +48,7 @@ public abstract class AbstractTransporter
public void peek( PeekTask task )
throws Exception
{
- Objects.requireNonNull( "task", "task cannot be null" );
+ Objects.requireNonNull( task, "task cannot be null" );
failIfClosed( task );
implPeek( task );
@@ -66,7 +66,7 @@ public abstract class AbstractTransporter
public void get( GetTask task )
throws Exception
{
- Objects.requireNonNull( "task", "task cannot be null" );
+ Objects.requireNonNull( task, "task cannot be null" );
failIfClosed( task );
implGet( task );
@@ -148,7 +148,7 @@ public abstract class AbstractTransporter
public void put( PutTask task )
throws Exception
{
- Objects.requireNonNull( "task", "task cannot be null" );
+ Objects.requireNonNull( task, "task cannot be null" );
failIfClosed( task );
implPut( task );
diff --git
a/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporterFactory.java
b/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporterFactory.java
index 5f5b48d..9ed2147 100644
---
a/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporterFactory.java
+++
b/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporterFactory.java
@@ -82,8 +82,8 @@ public final class ClasspathTransporterFactory
public Transporter newInstance( RepositorySystemSession session,
RemoteRepository repository )
throws NoTransporterException
{
- Objects.requireNonNull( "session", "session cannot be null" );
- Objects.requireNonNull( "repository", "repository cannot be null" );
+ Objects.requireNonNull( session, "session cannot be null" );
+ Objects.requireNonNull( repository, "repository cannot be null" );
return new ClasspathTransporter( session, repository );
}
diff --git
a/maven-resolver-transport-file/src/main/java/org/eclipse/aether/transport/file/FileTransporterFactory.java
b/maven-resolver-transport-file/src/main/java/org/eclipse/aether/transport/file/FileTransporterFactory.java
index 42e651f..349b436 100644
---
a/maven-resolver-transport-file/src/main/java/org/eclipse/aether/transport/file/FileTransporterFactory.java
+++
b/maven-resolver-transport-file/src/main/java/org/eclipse/aether/transport/file/FileTransporterFactory.java
@@ -69,8 +69,8 @@ public final class FileTransporterFactory
public Transporter newInstance( RepositorySystemSession session,
RemoteRepository repository )
throws NoTransporterException
{
- Objects.requireNonNull( "session", "session cannot be null" );
- Objects.requireNonNull( "repository", "repository cannot be null" );
+ Objects.requireNonNull( session, "session cannot be null" );
+ Objects.requireNonNull( repository, "repository cannot be null" );
return new FileTransporter( repository );
}
diff --git
a/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporter.java
b/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporter.java
index 9a1a3f7..4a98563 100644
---
a/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporter.java
+++
b/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporter.java
@@ -409,7 +409,7 @@ final class WagonTransporter
private void execute( TransportTask task, TaskRunner runner )
throws Exception
{
- Objects.requireNonNull( "task", "task cannot be null" );
+ Objects.requireNonNull( task, "task cannot be null" );
if ( closed.get() )
{
diff --git
a/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporterFactory.java
b/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporterFactory.java
index a8487d6..9ef3ecd 100644
---
a/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporterFactory.java
+++
b/maven-resolver-transport-wagon/src/main/java/org/eclipse/aether/transport/wagon/WagonTransporterFactory.java
@@ -115,8 +115,8 @@ public final class WagonTransporterFactory
public Transporter newInstance( RepositorySystemSession session,
RemoteRepository repository )
throws NoTransporterException
{
- Objects.requireNonNull( "session", "session cannot be null" );
- Objects.requireNonNull( "repository", "repository cannot be null" );
+ Objects.requireNonNull( session, "session cannot be null" );
+ Objects.requireNonNull( repository, "repository cannot be null" );
return new WagonTransporter( wagonProvider, wagonConfigurator,
repository, session );
}
