On 21/10/2016 22:07, Alexandre (Shura) Iline wrote:

Alan,

This is a newer version:
http://cr.openjdk.java.net/~shurailine/8159523/webrev.03/

The usages looks good in this version, would be interesting to get other opinions (although this isn't specifically module options, this is really more about replacing usages of ProcessTools).

One small comment is that I'm not sure about Task defining ALL-UNNAMED. It might be simpler to leave addExports taking two options so that the LHS is more obvious in the tests that use.

-Alan

Reply via email to