Thanks for the review. I don't mind filing a bug for javadoc change and
it's easy as it's a separate issue and cleaner to separate them. I'm
happy this saves me the hassle of a separate review, thanks!
Mandy
On 12/15/14 12:56 AM, Erik Joelsson wrote:
Looks good to me. I don't mind if you fix javadoc in the same change.
It's such a small issue I rather not go through the hassle of creating
a bug and a new review.
Thanks!
/Erik
On 2014-12-12 19:25, Mandy Chung wrote:
Webrev:
http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8067360/webrev.00/
This patch adds back the verify-modules target to the dependences of
the images target that got dropped in the merge and hides a bug in
jdeps -Xverify:access.
Erik - I notice make/Javadoc.gmk that sets -bootclasspath
$(JDK_OUTPUTDIR)/classes that does not exist. I include a patch to
make/Javadoc.gmk in the webrev. Do you prefer to separate it and
file a new bug for it?
Mandy
