On 19/12/2016 01:44, Mandy Chung wrote:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8168836/webrev.00/
This patch improves the warning/error message to include the option name, emit
a warning if unknown module is specified with —-patch-module be consistent with
the options.
The implementation update looks good. As --patch-modules is a slow path
then you could use cf.findModule(mn).ifPresent(...).
ExplodedModuleBuilder is useful infrastructure but I'm not sure that
test/tools/lib is the right place as that location seems to be for
jimage infrastructure (I've often thought this should move). Maybe
test/lib/testlibrary would be better because that there is where
ModuleUtils and the other test infrastructure used by these tests is.
Also just wondering if there is a better name for this class as it
supports both creation and compilation, maybe it should be in
ModuleUtils as Builder or ModuleMaker (you might have tried a few names
already). Once this is in then we might look at changes some of the
existing tests to use it.
-Alan
