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/04/11 20:25:27 UTC

[maven-indexer] 04/05: [MINDEXER-124] Add null checks to constructors

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

slachiewicz pushed a commit to branch MINDEXER-124
in repository https://gitbox.apache.org/repos/asf/maven-indexer.git

commit ce5391626fd987cb4ce148d3eec92a85fe59a286
Author: sixcorners <si...@gmail.com>
AuthorDate: Sun Feb 2 20:00:44 2020 -0600

    [MINDEXER-124] Add null checks to constructors
---
 .../java/org/apache/maven/index/reader/resource/BufferedResource.java   | 2 ++
 .../org/apache/maven/index/reader/resource/BufferedResourceHandler.java | 2 ++
 .../maven/index/reader/resource/BufferedWritableResourceHandler.java    | 2 ++
 .../org/apache/maven/index/reader/resource/PathWritableResource.java    | 2 ++
 .../apache/maven/index/reader/resource/PathWritableResourceHandler.java | 2 ++
 .../java/org/apache/maven/index/reader/resource/UriResourceHandler.java | 2 ++
 .../main/java/org/apache/maven/index/reader/resource/UrlResource.java   | 2 ++
 7 files changed, 14 insertions(+)

diff --git a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResource.java b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResource.java
index 5312833..a85872b 100644
--- a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResource.java
+++ b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResource.java
@@ -3,6 +3,7 @@ package org.apache.maven.index.reader.resource;
 import java.io.BufferedInputStream;
 import java.io.IOException;
 import java.io.InputStream;
+import java.util.Objects;
 import org.apache.maven.index.reader.ResourceHandler.Resource;
 
 /**
@@ -12,6 +13,7 @@ public class BufferedResource implements Resource {
   private final Resource resource;
 
   public BufferedResource(Resource resource) {
+    Objects.requireNonNull(resource, "resource cannot be null");
     this.resource = resource;
   }
 
diff --git a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResourceHandler.java b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResourceHandler.java
index f3269dc..b994295 100644
--- a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResourceHandler.java
+++ b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedResourceHandler.java
@@ -2,6 +2,7 @@ package org.apache.maven.index.reader.resource;
 
 import java.io.BufferedInputStream;
 import java.io.IOException;
+import java.util.Objects;
 import org.apache.maven.index.reader.ResourceHandler;
 
 /**
@@ -12,6 +13,7 @@ public class BufferedResourceHandler implements ResourceHandler {
   private final ResourceHandler resourceHandler;
 
   public BufferedResourceHandler(ResourceHandler resourceHandler) {
+    Objects.requireNonNull(resourceHandler, "resourceHandler cannot be null");
     this.resourceHandler = resourceHandler;
   }
 
diff --git a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedWritableResourceHandler.java b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedWritableResourceHandler.java
index 67ff338..26eae6a 100644
--- a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedWritableResourceHandler.java
+++ b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/BufferedWritableResourceHandler.java
@@ -3,6 +3,7 @@ package org.apache.maven.index.reader.resource;
 import java.io.BufferedInputStream;
 import java.io.BufferedOutputStream;
 import java.io.IOException;
+import java.util.Objects;
 import org.apache.maven.index.reader.WritableResourceHandler;
 
 /**
@@ -13,6 +14,7 @@ public class BufferedWritableResourceHandler implements WritableResourceHandler
   private final WritableResourceHandler writableResourceHandler;
 
   public BufferedWritableResourceHandler(WritableResourceHandler writableResourceHandler) {
+    Objects.requireNonNull(writableResourceHandler, "writableResourceHandler cannot be null");
     this.writableResourceHandler = writableResourceHandler;
   }
 
diff --git a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResource.java b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResource.java
index 21659ac..6e1a8a3 100644
--- a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResource.java
+++ b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResource.java
@@ -6,6 +6,7 @@ import java.io.OutputStream;
 import java.nio.file.Files;
 import java.nio.file.NoSuchFileException;
 import java.nio.file.Path;
+import java.util.Objects;
 import org.apache.maven.index.reader.WritableResourceHandler.WritableResource;
 
 /**
@@ -15,6 +16,7 @@ public class PathWritableResource implements WritableResource {
   private final Path path;
 
   public PathWritableResource(Path path) {
+    Objects.requireNonNull(path, "path cannot be null");
     this.path = path;
   }
 
diff --git a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResourceHandler.java b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResourceHandler.java
index e98b13b..760a9dc 100644
--- a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResourceHandler.java
+++ b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/PathWritableResourceHandler.java
@@ -2,6 +2,7 @@ package org.apache.maven.index.reader.resource;
 
 import java.io.IOException;
 import java.nio.file.Path;
+import java.util.Objects;
 import org.apache.maven.index.reader.WritableResourceHandler;
 
 /**
@@ -11,6 +12,7 @@ public class PathWritableResourceHandler implements WritableResourceHandler {
   private final Path path;
 
   public PathWritableResourceHandler(Path path) {
+    Objects.requireNonNull(path, "path cannot be null");
     this.path = path;
   }
 
diff --git a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UriResourceHandler.java b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UriResourceHandler.java
index cce1b00..561177c 100644
--- a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UriResourceHandler.java
+++ b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UriResourceHandler.java
@@ -2,6 +2,7 @@ package org.apache.maven.index.reader.resource;
 
 import java.io.IOException;
 import java.net.URI;
+import java.util.Objects;
 import org.apache.maven.index.reader.ResourceHandler;
 
 /**
@@ -11,6 +12,7 @@ public class UriResourceHandler implements ResourceHandler {
   private final URI uri;
 
   public UriResourceHandler(URI uri) {
+    Objects.requireNonNull(uri, "uri cannot be null");
     this.uri = uri;
   }
 
diff --git a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UrlResource.java b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UrlResource.java
index 584ef40..c2e215c 100644
--- a/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UrlResource.java
+++ b/indexer-reader/src/main/java/org/apache/maven/index/reader/resource/UrlResource.java
@@ -4,6 +4,7 @@ import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.InputStream;
 import java.net.URL;
+import java.util.Objects;
 import org.apache.maven.index.reader.ResourceHandler.Resource;
 
 /**
@@ -13,6 +14,7 @@ public class UrlResource implements Resource {
   private final URL url;
 
   public UrlResource(URL url) {
+    Objects.requireNonNull(url, "url cannot be null");
     this.url = url;
   }