The updated webrev looks good.
--Sean
On 07/10/2014 01:59 AM, Wang Weijun wrote:
Updated webrev at
http://cr.openjdk.java.net/~weijun/8043071/webrev.01/
All your suggestions accepted, plus
- New test on new classes added into KerberosHashEqualsTest.java
- A duplicate test/sun/security/krb5/auto/KerberosHashEqualsTest.java removed
- I don't know why KerberosKey allows a null principal, but anyway update its
toString() to avoid NPE
- KerberosTicket.toString() now returns "Destroyed KerberosTicket" instead of
throwing an ISE, to be consistent with other classes
Thanks
Max