On 8/23/17 4:02 PM, Jonathan Gibbons wrote:
Removed ArrayDeque from patch.
Updated webrev and API

Webrev: http://cr.openjdk.java.net/~jjg/8186684/webrev.01/index.html
API: http://cr.openjdk.java.net/~jjg/8186684/api.01/overview-summary.html

src/java.base/share/classes/java/lang/module/Configuration.java

- * resolution</a> or resolution with <a href="#service-binding">service binding</a>.
+ * resolution</a> or resolution with
+ * <a href="{@docRoot}/java/lang/Configuration#service-binding">service binding</a>.

This link is still broken (should be java/lang/module/Configuration.html) Does javadoc detect it?

I also like the javadoc enhancement providing a way to reference a specified id other than a class member.

Other changes look okay.

Mandy


Reply via email to