On 06/10/2016 23:17, Mandy Chung wrote:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8162401/webrev.00/
Currently, `--add-exports` and `--add-reads` options do not accept duplicated
module/package and it is an error to attempt to export the same package more
than once on the command-line. There are cases where allow augmenting the
command-line options to add `--add-exports` or `--add-reads`. jtreg is one
example that currently parses the command-line options and has to merge
properly if there exists a `--add-exports` option of a given module/package.
This patch relaxes this restriction and allow multiple `--add-exports` and
`--add-reads` options with the same module/package. In addition, these 2
options allow unknown module/package and emit a warning rather than as a fatal
error, to be consistent with javac.
"Must be specified" might be clearer than "must not be empty". Otherwise
looks good. Do you mind pushing this to jigsaw/jake at the same time as
jdk9/dev? This is because there there are a few things that need
adjustment for --add-exports-private, mostly tests that aren't in jake yet.
-Alan