From 33de246d75bb3f0bbbf7ae137b34638fd1a06d19 Mon Sep 17 00:00:00 2001
From: BJ Hargrave <bj@hargrave.dev>
Date: Mon, 23 May 2022 09:38:48 -0400
Subject: [PATCH] io: Make FileTree predicate methods public since they can be
 useful

I wanted to use these methods on a FileTree object, but they were
private.

Signed-off-by: BJ Hargrave <bj@hargrave.dev>
---
 aQute.libg/src/aQute/lib/io/FileTree.java     | 26 ++++++++++++++++---
 aQute.libg/src/aQute/lib/io/package-info.java |  2 +-
 2 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/aQute.libg/src/aQute/lib/io/FileTree.java b/aQute.libg/src/aQute/lib/io/FileTree.java
index 797a86c594..5a26ea8b89 100644
--- a/aQute.libg/src/aQute/lib/io/FileTree.java
+++ b/aQute.libg/src/aQute/lib/io/FileTree.java
@@ -1,5 +1,7 @@
 package aQute.lib.io;
 
+import static java.util.Objects.requireNonNull;
+
 import java.io.File;
 import java.nio.file.Path;
 import java.text.CollationKey;
@@ -103,7 +105,16 @@ public List<File> getFiles(File baseDir, List<String> defaultIncludes) {
 		return getFiles(baseDir, files.isEmpty() ? paths.matches(defaultIncludes) : paths.matches());
 	}
 
-	private List<File> getFiles(File baseDir, Predicate<String> matches) {
+	/**
+	 * Return a list of files using the specified baseDir and the specified
+	 * relative path matching predicate.
+	 *
+	 * @param baseDir The base directory for locating files.
+	 * @param matches The path matching predicate. The predicate only matches
+	 *            against the relative path from the specified baseDir.
+	 * @return A list of files.
+	 */
+	public List<File> getFiles(File baseDir, Predicate<String> matches) {
 		ArrayList<File> result = new ArrayList<>();
 		new FileTreeSpliterator(baseDir, matches, files).forEachRemaining(result::add);
 		result.trimToSize();
@@ -136,7 +147,16 @@ public Stream<File> stream(File baseDir, List<String> defaultIncludes) {
 		return stream(baseDir, files.isEmpty() ? paths.matches(defaultIncludes) : paths.matches());
 	}
 
-	private Stream<File> stream(File baseDir, Predicate<String> matches) {
+	/**
+	 * Return a stream of files using the specified baseDir and the specified
+	 * relative path matching predicate.
+	 *
+	 * @param baseDir The base directory for locating files.
+	 * @param matches The path matching predicate. The predicate only matches
+	 *            against the relative path from the specified baseDir.
+	 * @return A stream of files.
+	 */
+	public Stream<File> stream(File baseDir, Predicate<String> matches) {
 		return StreamSupport.stream(new FileTreeSpliterator(baseDir, matches, files), false);
 	}
 
@@ -154,7 +174,7 @@ final static class FileTreeSpliterator extends AbstractSpliterator<File> {
 			super(Long.MAX_VALUE,
 				Spliterator.ORDERED | Spliterator.IMMUTABLE | Spliterator.DISTINCT | Spliterator.NONNULL);
 			basePath = baseDir.toPath();
-			this.matches = matches;
+			this.matches = requireNonNull(matches);
 			List<File> copy = new ArrayList<>(files); // mutable copy
 			this.files = copy;
 			extra = copy.spliterator(); // late-binding
diff --git a/aQute.libg/src/aQute/lib/io/package-info.java b/aQute.libg/src/aQute/lib/io/package-info.java
index 58c91fe432..8eb8fed15a 100644
--- a/aQute.libg/src/aQute/lib/io/package-info.java
+++ b/aQute.libg/src/aQute/lib/io/package-info.java
@@ -1,4 +1,4 @@
-@Version("4.2.0")
+@Version("4.3.0")
 package aQute.lib.io;
 
 import org.osgi.annotation.versioning.Version;