Mandy,

webrev has been updated accordingly.

http://cr.openjdk.java.net/~sherman/8167328/

Thanks,
Sherman

On 12/01/2016 11:58 AM, Mandy Chung wrote:
On Nov 30, 2016, at 4:48 PM, Xueming Shen<xueming.s...@oracle.com>  wrote:

Hi,

Please help review the change for #8167328

issue: https://bugs.openjdk.java.net/browse/JDK-8167328
webrev: http://cr.openjdk.java.net/~sherman/8167328

It appears the desired behavior is to throw the error + help msg if the
option "--print-module-descriptor"/ "-d" is specified with some extra
file names.
This looks good.

It may be useful to include the arguments in the error message:

   48         '-d, --print-module-descriptor' option requires no input files to 
be specified!

Mandy


Reply via email to