Re: [8] Request for review: 8019539: Fix doclint errors in java.security and its subpackages

[Jason Uh]

Joe,

Thanks for your comments. I just sent out an updated webrev, but let me 
put out a new one including my updates and changes to address your comments.

Thanks,
Jason

On 07/01/2013 04:16 PM, Joseph Darcy wrote:

On 7/1/2013 3:22 PM, Jason Uh wrote:
Joe,

Could I please get a review of this changeset?

These changes fixed all (40) doclint errors in java.security and its
subpackages.

Webrev: http://cr.openjdk.java.net/~juh/8019539/webrev.00/

No undesirable changes were detected by specdiff.

Thanks,
Jason

In src/share/classes/java/security/Security.java, instead of

      495      * <li>
<i>&lt;crypto_service&gt;.&lt;algorithm_or_type&gt;</i> <p> The

please see if something like

      495      * <li> <i>{@literal
<crypto_service>.<algorithm_or_type>}</i> <p> The

will work.

In src/share/classes/java/security/cert/X509CRL.java, rather than replacing

416      * signatureAlgorithm   AlgorithmIdentifier<p>

with

  417      * signatureAlgorithm   AlgorithmIdentifier
  418      * </pre><pre>

can the trailing "<p>" just be removed? Same comment for
X509CRLEntry.java and X509Certificate.java.

If you are not tied to a particular ASN.1 syntax, I've used HTML
definition lists for this purpose; there are samples in java.lang.Integer.

Thanks,

-Joe