[15] RFR 8241761 : Typos: empty lines in javadoc, inconsistent indents, etc. (security-libs only)
Weijun Wang
weijun.wang at oracle.com
Wed Apr 1 09:59:36 UTC 2020
Looks fine to me.
If I have to pick one tiny problem, you can remove a blank in line 58 below to make the indentation more precise:
src/java.security.sasl/share/classes/javax/security/sasl/SaslServer.java:
57 * status = ldap.sendBindResponse(mechanism, challenge,
58 * SASL_BIND_IN_PROGRESS);
Thanks,
Max
> On Mar 31, 2020, at 5:39 AM, Ivan Gerasimov <ivan.gerasimov at oracle.com> wrote:
>
> Hello!
>
> The fix follows up on JDK-8241727 [1].
>
> This is a javadoc/comments only fix in the security-libs area.
>
> The changes are to remove redundant empty lines, correct indentation, or otherwise restore harmony.
>
> Would you please help review this rather technical fix?
>
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8241761
> WEBREV: http://cr.openjdk.java.net/~igerasim/8241761/00/webrev/
>
> Thank in advance!
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8241727
>
> --
> With kind regards,
> Ivan Gerasimov
>
More information about the security-dev
mailing list