Looks fine to me. Thanks, Xuelei
On 6/29/2014 7:59 AM, Jamil Nimeh wrote: > Hello all, > > This follow-on webrev addresses the following issues: > > * Adds braces to if/else constructs > * Fixes imports in Serial.java and Generic.java tests to explicitly > import javax.security.auth.Subject rather than > javax.security.auth.*. The overridden Subject.java solution in the > test directory appears to cause compilation issues, but specifying > the Subject class on the import line fixes things, and the tests > don't need all of javax.security.auth. > * Added the hashCode method to the SecureSet inner class. > > http://cr.openjdk.java.net/~ascarpino/8015081/webrev.08 > > Thanks, > --Jamil >
