Hi again, thanks for valuable feedback - I missed those before. Second version of a patch: http://cr.openjdk.java.net/~mkos/8130753/jaxws.02/ issues addressed: * <tt> : not backporting correctly changes done in jdk repo, will fix in standalone repos * <pre>/ {@code : added <pre> again, in two cases used <pre><code> * copyright years : fixed Otherwise I don't use -Xdoclint but I generate javadoc so I see warnings - before patch: 10 errors 78 warnings after patch: 78 warnings I also used specdiff to doblecheck that there are no differences in output javadoc - those are classes defined by JSRs, so no changes are wanted without MR. Only 2 changes found (typos). Additional issues (warnings) would be addressed during MRs. Thanks Miran On 08/07/15 18:36, joe darcy wrote:
Hi Miroslav,
At least some of the changes in the patch are going in the wrong direction. There are many instances toward the start of the patch where {@code Foo} is listed as being replaced by <tt>Foo</tt>, but {@code Foo} is what we want to use now.
Are you running doclint over Java source or in a docs build to verify the changes?
Thanks,
-Joe
On 7/8/2015 5:29 AM, Miroslav Kos wrote:
Hi everybody, I'd like to ask for review for following issue:
JBS: 8130753: Sync-up javadoc changes in jax-ws area - includes JAX-B API, JAX-WS API, SAAJ-API webrev: http://cr.openjdk.java.net/~mkos/8130753/jaxws.01/
It addresses all the javadoc errors in all 3 JSRs: SAAJ, JAX-B and JAX-WS and it is also syncup with standalone repos. This clean-up had to be done in order to proceed with changes necessary for Jigsaw (specifically SAAJ API and possibly JAX-WS).
Thanks Miran