> On Mar 4, 2017, at 5:14 PM, Kevin Rushforth <kevin.rushfo...@oracle.com> > wrote: > > http://cr.openjdk.java.net/~kcr/8170701/webrev.01/ >
40 * object {@link Module#isOpen opens} the containing package to the Nit: s/@link/@linkplain 41 * {@code javafx.fxml} module, either in its {@link ModuleDescriptor} 42 * (module-info.class) or by calling {@link Module#addOpens}. Do you intend to take out β(module-info.class)β? 43 * An object is also reflectively accessible if it is declared as public, Does βitβ mean its constructor? 44 * is in a public class, and the module containing that class 45 * {@link Module#isExported(String,Module) exports} Nit: s/@link/@linkplain 46 * the containing package to the {@code javafx.fxml} module. This is word-smithing and formatting nit. No need to send a new webrev. Mandy