Makes sense to me. The changes look fine. -Chris
> On 31 Dec 2013, at 02:07, Joe Darcy <joe.da...@oracle.com> wrote: > > Hello, > > Starting in JDK 8, there have been efforts to clean up the doclint warnings > and errors, especially for types going into the coredocs bundle. The core > libraries are doclint clean and much progress has been made on the client > area. I'll soon have a patch out for review that will clear all the doclint > problems in three of the five doclint categories in client, meaning several > categories of doclint checks will be able to be turned on for the whole core > docs bundle :-) > > However, not all of the sources used to build the ~15 docs bundles have been > cleaned to the same degree and they should be expected to be cleaned at > different rates. Therefore, until all the bundles have been cleared of all > problems, I think the commands to make each bundle should have its own > doclint option so they can be tightened independently. > > To be clear, the goal would be for "-Xdoclint:all/protected" to be used on > all bundles. > > In the mean time, please review the patch below which introduces a separate > doclint option for each bundle. > > Thanks, > > -Joeg > > diff -r bd254db01a0e make/Javadoc.gmk > --- a/make/Javadoc.gmk Thu Dec 19 14:11:12 2013 +0100 > +++ b/make/Javadoc.gmk Mon Dec 30 18:06:02 2013 -0800 > @@ -265,7 +265,6 @@ > -quiet \ > -use \ > -keywords \ > - -Xdoclint:none \ > -Xprofilespath $(JDK_TOPDIR)/make/profile-rtjar-includes.txt \ > $(ADDITIONAL_JAVADOCFLAGS) > > @@ -381,6 +380,7 @@ > $(COREAPI_OPTIONS_FILE): $(COREAPI_OVERVIEW) > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ISO-8859-1) ; \ > $(call OptionPair,-tag,beaninfo:X) ; \ > @@ -456,6 +456,7 @@ > $(DOCLETAPI_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-breakiterator) ; \ > @@ -512,6 +513,7 @@ > $(TAGLETAPI_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nonavbar) ; \ > @@ -563,6 +565,7 @@ > $(DOMAPI_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-splitIndex) ; \ > @@ -625,6 +628,7 @@ > $(JDI_OPTIONS_FILE): $(JDI_OVERVIEW) > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionPair,-overview,$(JDI_OVERVIEW)) ; \ > @@ -712,6 +716,7 @@ > $(JAAS_OPTIONS_FILE): $(JAAS_OVERVIEW) > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionPair,-overview,$(JAAS_OVERVIEW)) ; \ > @@ -764,6 +769,7 @@ > $(JGSS_OPTIONS_FILE): $(JGSS_OVERVIEW) > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nodeprecatedlist) ; \ > @@ -816,6 +822,7 @@ > $(SMARTCARDIO_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nodeprecatedlist) ; \ > @@ -866,6 +873,7 @@ > $(HTTPSERVER_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nodeprecatedlist) ; \ > @@ -926,6 +934,7 @@ > $(MGMT_OPTIONS_FILE): $(MGMT_OVERVIEW) > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nodeprecatedlist) ; \ > @@ -977,6 +986,7 @@ > $(ATTACH_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nodeprecatedlist) ; \ > @@ -1027,6 +1037,7 @@ > $(JCONSOLE_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nodeprecatedlist) ; \ > @@ -1079,6 +1090,7 @@ > $(TREEAPI_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionPair,-doctitle,$(TREEAPI_DOCTITLE)) ; \ > @@ -1130,6 +1142,7 @@ > $(SCTPAPI_OPTIONS_FILE): > $(prep-target) > @($(call OptionOnly,$(COMMON_JAVADOCFLAGS)) ; \ > + $(call OptionOnly,-Xdoclint:none) ; \ > $(call OptionPair,-sourcepath,$(RELEASEDOCS_SOURCEPATH)) ; \ > $(call OptionPair,-encoding,ascii) ; \ > $(call OptionOnly,-nodeprecatedlist) ; \ >