On 6/10/16 9:31 AM, Mandy Chung wrote:
Harold has incorporated feedback from David, Coleen, Lois and others and updated webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8136930/webrev.02/The only change compared to webrev.01 is in hotspot arguments.{cpp,hpp} and a new ModuleOptionsTest.java. I see concerns from Coleen on the option name and syntax. I’d like to separate that discussion from this fix and follow up after this push unless any objection.
Hmmmm... My reading of Coleen's comments is that she is objecting to adding these new options to the JVM. Her concerns are _not_ just on the option name and syntax. Please do not push this until Coleen is satisfied with the resolution to her objections. Dan
Mandy
