RFR: JDK-8319123 : Implementation of JEP-461: Stream Gatherers (Preview) [v2]

Per Minborg pminborg at openjdk.org
Thu Nov 9 08:57:15 UTC 2023


On Wed, 8 Nov 2023 21:08:03 GMT, Viktor Klang <vklang at openjdk.org> wrote:

>> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461)
>
> Viktor Klang has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Addressing review feedback
>  - Make Gatherer.andThen take a wildcard for the rhs Gatherer state type

src/java.base/share/classes/java/util/stream/Gatherers.java line 326:

> 324: 
> 325:     /**
> 326:      * Gathers elements into fixed-size windows. The last window may contain

Returns a ...

src/java.base/share/classes/java/util/stream/Gatherers.java line 347:

> 345:      * @throws IllegalArgumentException when windowSize is less than 1
> 346:      */
> 347:     public static <TR> Gatherer<TR, ?, List<TR>> windowFixed(int windowSize) {

In my opinion, it would be nicer to let `Gatherers` be a shopping window for cool gatherers. Ideally, I think only the docs, methods, parameters, and invariant assertions should be visible here. The rest could be tucked away under the covers.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16420#discussion_r1387675852
PR Review Comment: https://git.openjdk.org/jdk/pull/16420#discussion_r1387674874


More information about the core-libs-dev mailing list