On Thu, 13 May 2021 18:53:25 GMT, Mandy Chung <[email protected]> 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
