On 01/10/2012 01:35, Iris Clark wrote:
Hi, Alan.
Looks fantastic!
By fixing these javadoc warnings for @throws and @param, I'm assuming that we
will be changing the generated javadoc. Are any of these changes significant
enough to warrant a ccc? Most of the changes looked fairly innocuous but there
are a couple here and there (e.g. java.lang.management.ThreadInfo,
java.util.JapaneseImperialCalendar) that may warrant a closer look.
Thanks Irish. I don't believe there is anything significant. A lot of
the changes are actually to private or package-private methods so they
don't show up in the javadoc anyway (the changes to ThreadInfo and
JapaneseImperialCalendar are for private methods and a private constructor).
In java.util.Locale it looks like you determined that the '/' used for tag
self-termination is unacceptable. Does it need to stay there now that you've
done the explicit termination?
(e.g.
your new l86:<dt><a name="def_language"/><b>language</b></a></dt>
Proposed new:<dt><a name="def_language"><b>language</b></a></dt>
)
You're right, I shouldn't have touched those (the only reason I was in
Locale was because there's a typo in the @throws of the readObject method).
-Alan
