On 22/04/2017 16:42, Mandy Chung wrote:
Have an explicit list is another alternative. OTOH I think only deployment
modules will name with these words though which was what I initially want to
cover.
Since only 4 modules we are concerned about, I updated the patch to list the
ones needed to be excluded rather than a superset as you suggest:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8179025/webrev.01/
The updated filter looks okay to me.
In JdkQualifiedExportTest.accept then
"cf.findModule(target).orElse(null) == null" looks a bit odd (I assume
this is what promoted Peter bringing up isEmpty on core-libs-dev. There
are a dozen ways you could do this of course, only alternative is:
return cf.findModule(target).map(m -> false).orElse(true);
-Alan
