RFR: 8180352: Add Stream.toList() method

Paul Sandoz psandoz at openjdk.java.net
Fri Nov 6 02:57:56 UTC 2020


On Fri, 6 Nov 2020 02:50:29 GMT, Stuart Marks <smarks at openjdk.org> wrote:

>> test/jdk/java/util/stream/test/org/openjdk/tests/java/util/stream/ToListOpTest.java line 73:
>> 
>>> 71:     }
>>> 72: 
>>> 73:     @Test(dataProvider = "withNull:StreamTestData<Integer>", dataProviderClass = StreamTestDataProvider.class)
>> 
>> Given the non-default `toList()` implementation defers to the `toArray()` terminal op there is no need for this and the following tests, which are really designed to shake out the optimizations when producing and operating on arrays.
>
> OK, I'll remove all the tests starting from here to the end of the file. I'm assuming that's the set of tests you're referring to.

Yes from line 73 to <SHIFT>G (the end).

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

PR: https://git.openjdk.java.net/jdk/pull/1026


More information about the core-libs-dev mailing list