Looks good to me.

Xuelei

On 6/4/2020 8:18 PM, Weijun Wang wrote:
Please review the patch below. The tag name should be camelCased.

*diff --git a/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosPrincipal.java b/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosPrincipal.java* *--- a/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosPrincipal.java* *+++ b/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosPrincipal.java*
@@ -106,7 +106,7 @@
       *
       * <p>If the input name does not contain a realm, the default realm
       * is used. The default realm can be specified either in a Kerberos
-     * configuration file or via the {@systemproperty java.security.krb5.realm} +     * configuration file or via the {@systemProperty java.security.krb5.realm}
       * system property. For more information, see the
       * {@extLink security_guide_jgss_tutorial Kerberos Requirements}.
       *
@@ -155,7 +155,7 @@
       *
       * <p>If the input name does not contain a realm, the default realm
       * is used. The default realm can be specified either in a Kerberos
-     * configuration file or via the {@systemproperty java.security.krb5.realm} +     * configuration file or via the {@systemProperty java.security.krb5.realm}
       * system property. For more information, see the
       * {@extLink security_guide_jgss_tutorial Kerberos Requirements}.
       *

Thanks,
Max

Reply via email to