RFR: JDK-8263545: Convert jpackage to use Stream.toList()
Alexey Semenyuk
asemenyuk at openjdk.java.net
Mon Mar 15 17:36:15 UTC 2021
On Sun, 14 Mar 2021 18:22:50 GMT, Ian Graves <igraves at openjdk.org> 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 changes are simple substitutions.
Looks good. Minor improvements suggested.
src/jdk.jpackage/windows/classes/jdk/jpackage/internal/WinMsiBundler.java line 200:
> 198: ).map(e -> {
> 199: e.getValue().setOutputFileName(e.getKey());
> 200: return (WixFragmentBuilder) e.getValue();
Why this explicit cast is needed here?
src/jdk.jpackage/share/classes/jdk/jpackage/internal/DottedVersion.java line 151:
> 149: components.add(BigInteger.ZERO);
> 150: }
> 151: return components.stream().toList().toArray(BigInteger[]::new);
I guess this can be simplified down to `components.stream().toArray(BigInteger[]::new);`
-------------
PR: https://git.openjdk.java.net/jdk/pull/2997
More information about the core-libs-dev
mailing list