You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by cs...@apache.org on 2022/02/19 18:46:40 UTC
[maven-resolver] 01/01: Fix bad requireNonNull checks, part2
This is an automated email from the ASF dual-hosted git repository.
cstamas pushed a commit to branch fix-bad-requireNonNull-checks-pt2
in repository https://gitbox.apache.org/repos/asf/maven-resolver.git
commit 97b897c2ab4699885bf140a89010f2125601f36b
Author: Tamas Cservenak <ta...@cservenak.net>
AuthorDate: Sat Feb 19 19:46:17 2022 +0100
Fix bad requireNonNull checks, part2
---
.../aether/connector/basic/BasicRepositoryConnectorFactory.java | 4 ++--
.../org/eclipse/aether/internal/impl/DefaultTransporterProvider.java | 4 ++--
.../org/eclipse/aether/transport/http/HttpTransporterFactory.java | 4 ++--
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/maven-resolver-connector-basic/src/main/java/org/eclipse/aether/connector/basic/BasicRepositoryConnectorFactory.java b/maven-resolver-connector-basic/src/main/java/org/eclipse/aether/connector/basic/BasicRepositoryConnectorFactory.java
index 280df6d..576d121 100644
--- a/maven-resolver-connector-basic/src/main/java/org/eclipse/aether/connector/basic/BasicRepositoryConnectorFactory.java
+++ b/maven-resolver-connector-basic/src/main/java/org/eclipse/aether/connector/basic/BasicRepositoryConnectorFactory.java
@@ -179,8 +179,8 @@ public final class BasicRepositoryConnectorFactory
public RepositoryConnector newInstance( RepositorySystemSession session, RemoteRepository repository )
throws NoRepositoryConnectorException
{
- requireNonNull( "session", "session cannot be null" );
- requireNonNull( "repository", "repository cannot be null" );
+ requireNonNull( session, "session cannot be null" );
+ requireNonNull( repository, "repository cannot be null" );
return new BasicRepositoryConnector( session, repository, transporterProvider, layoutProvider,
checksumPolicyProvider, fileProcessor, providedChecksumsSources );
diff --git a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultTransporterProvider.java b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultTransporterProvider.java
index af1d1d4..07ba820 100644
--- a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultTransporterProvider.java
+++ b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultTransporterProvider.java
@@ -90,8 +90,8 @@ public final class DefaultTransporterProvider
public Transporter newTransporter( RepositorySystemSession session, RemoteRepository repository )
throws NoTransporterException
{
- requireNonNull( "session", "session cannot be null" );
- requireNonNull( "repository", "repository cannot be null" );
+ requireNonNull( session, "session cannot be null" );
+ requireNonNull( repository, "repository cannot be null" );
PrioritizedComponents<TransporterFactory> factories = new PrioritizedComponents<>( session );
for ( TransporterFactory factory : this.factories )
diff --git a/maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporterFactory.java b/maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporterFactory.java
index 7eff428..0143035 100644
--- a/maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporterFactory.java
+++ b/maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporterFactory.java
@@ -98,8 +98,8 @@ public final class HttpTransporterFactory
public Transporter newInstance( RepositorySystemSession session, RemoteRepository repository )
throws NoTransporterException
{
- requireNonNull( "session", "session cannot be null" );
- requireNonNull( "repository", "repository cannot be null" );
+ requireNonNull( session, "session cannot be null" );
+ requireNonNull( repository, "repository cannot be null" );
return new HttpTransporter( extractors, repository, session );
}