You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2021/01/12 16:47:30 UTC
[commons-io] branch master updated: Use Objects.requireNonNull().
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-io.git
The following commit(s) were added to refs/heads/master by this push:
new 817bdf1 Use Objects.requireNonNull().
817bdf1 is described below
commit 817bdf10eeceeda08a8fc30d80b67b94e1baa754
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Tue Jan 12 11:38:46 2021 -0500
Use Objects.requireNonNull().
---
src/main/java/org/apache/commons/io/FilenameUtils.java | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/main/java/org/apache/commons/io/FilenameUtils.java b/src/main/java/org/apache/commons/io/FilenameUtils.java
index bd92853..04a3a93 100644
--- a/src/main/java/org/apache/commons/io/FilenameUtils.java
+++ b/src/main/java/org/apache/commons/io/FilenameUtils.java
@@ -24,6 +24,7 @@ import java.util.Arrays;
import java.util.Collection;
import java.util.Deque;
import java.util.List;
+import java.util.Objects;
import java.util.regex.Matcher;
import java.util.regex.Pattern;
@@ -537,11 +538,7 @@ public class FilenameUtils {
*/
public static boolean directoryContains(final String canonicalParent, final String canonicalChild)
throws IOException {
-
- // Fail fast against NullPointerException
- if (canonicalParent == null) {
- throw new IllegalArgumentException("Directory must not be null");
- }
+ Objects.requireNonNull(canonicalParent, "canonicalParent");
if (canonicalChild == null) {
return false;