Remove concat implementation (#29)

+ The JDK has this, and concatenating multiple streams is not enough of a difference to keep this.
tginsberg · Sep 23, 2024 · 1fc1be7 · 1fc1be7
1 parent c451c10
commit 1fc1be7
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 173 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,6 @@
+### 0.4.0
++ Remove `concat` implementation (the JDK has this)
+
 ### 0.3.0
 + Move minimum Java version from 22 to 23
 

diff --git a/src/main/java/com/ginsberg/gatherers4j/ConcatenationGatherer.java b/src/main/java/com/ginsberg/gatherers4j/ConcatenationGatherer.java
diff --git a/src/main/java/com/ginsberg/gatherers4j/Gatherers4j.java b/src/main/java/com/ginsberg/gatherers4j/Gatherers4j.java
@@ -27,15 +27,6 @@
 
 public class Gatherers4j {
 
-    /**
-     * Concatenate the given <code>Stream&lt;INPUT&gt;</code> to the end of the current stream, in order.
-     *
-     * @param concatThis A non-null <code>Stream&lt;INPUT&gt;</code> instance to concatenate.
-     */
-    public static <INPUT> ConcatenationGatherer<INPUT> concat(final Stream<INPUT> concatThis) {
-        return new ConcatenationGatherer<>(concatThis);
-    }
-
     /**
      * Limit the number of elements in the stream to some number per period, dropping anything over the
      * limit during the period.

diff --git a/src/test/java/com/ginsberg/gatherers4j/ConcatenationGathererTest.java b/src/test/java/com/ginsberg/gatherers4j/ConcatenationGathererTest.java