Re: RFR: JDK-8263545: Convert jpackage to use Stream.toList() [v2]

2021-03-15 Thread Alexander Matveev
On Mon, 15 Mar 2021 18:53:26 GMT, Ian Graves wrote: >> This converts jpackage to use `Stream.toList()` instead of >> `Stream.collect(Collectors.toList())`. One piece of code was modified to not >> mutate a list in addition to one test that used a mutating sort on a list. >> The rest of the

Re: RFR: JDK-8263545: Convert jpackage to use Stream.toList() [v2]

2021-03-15 Thread Alexey Semenyuk
On Mon, 15 Mar 2021 18:53:26 GMT, Ian Graves wrote: >> This converts jpackage to use `Stream.toList()` instead of >> `Stream.collect(Collectors.toList())`. One piece of code was modified to not >> mutate a list in addition to one test that used a mutating sort on a list. >> The rest of the

Re: RFR: JDK-8263545: Convert jpackage to use Stream.toList() [v2]

2021-03-15 Thread Ian Graves
On Mon, 15 Mar 2021 17:29:46 GMT, Alexey Semenyuk wrote: >> Ian Graves has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Clarifying type argument and shorting a toArray invocation > >

Re: RFR: JDK-8263545: Convert jpackage to use Stream.toList() [v2]

2021-03-15 Thread Ian Graves
> This converts jpackage to use `Stream.toList()` instead of > `Stream.collect(Collectors.toList())`. One piece of code was modified to not > mutate a list in addition to one test that used a mutating sort on a list. > The rest of the changes are simple substitutions. Ian Graves has updated