RFR: 8266461: tools/jmod/hashes/HashesTest.java fails: static @Test methods [v3]
Lance Andersen
lancea at openjdk.java.net
Thu May 13 19:30:38 UTC 2021
On Thu, 13 May 2021 18:53:25 GMT, Mandy Chung <mchung at openjdk.org> wrote:
>> Lance Andersen has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update copyright typo
>
> test/jdk/tools/jmod/hashes/HashesTest.java line 90:
>
>> 88: }
>> 89: this.mods = dest.resolve("mods");
>> 90: Path srcDir = dest.resolve("src");
>
> you can just get rid of this local variable and do this in line 92:
>
> this.builder = new ModuleInfoMaker(dest.resolve("src"));
Yes I can do that and realized that after I pushed the update
> test/jdk/tools/jmod/hashes/HashesTest.java line 387:
>
>> 385: if (hashes != null) {
>> 386: hashes.names().stream().sorted().forEach(n ->
>> 387: System.out.format(" %s %s%n", n, toHex(hashes.hashFor(n)))
>
> Nit: the extra whitespaces in line 384 and 387 may be added by IDE. Can you revert it.
Intellij did that. I can tweak
-------------
PR: https://git.openjdk.java.net/jdk/pull/4009
More information about the core-libs-dev
mailing list