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:39 UTC

[maven-resolver] branch fix-bad-requireNonNull-checks-pt2 created (now 97b897c)

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

cstamas pushed a change to branch fix-bad-requireNonNull-checks-pt2
in repository https://gitbox.apache.org/repos/asf/maven-resolver.git.


      at 97b897c  Fix bad requireNonNull checks, part2

This branch includes the following new commits:

     new 97b897c  Fix bad requireNonNull checks, part2

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


[maven-resolver] 01/01: Fix bad requireNonNull checks, part2

Posted by cs...@apache.org.
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 );
     }