Re: 8178380: Module system implementation refresh (5/2017 update)

[Mandy Chung]

> On May 1, 2017, at 1:28 PM, Alan Bateman <alan.bate...@oracle.com> wrote:
> 
>    http://cr.openjdk.java.net/~alanb/8178380/2/

I reviewed all repos in the new version.

src/java.base/share/classes/jdk/internal/module/ModuleBootstrap.java

 148         if (System.getProperty("jdk.module.minimumBoot") != null) {
This property can be removed after read the value, if present.

 287         if (propValue != null && Boolean.getBoolean(propValue))

It should use Boolean.parseBoolean.

I have fixed the above injake and also added a new test to verify 
-—show-module-resolution option.

Otherwise, all looks good.
Mandy