New webrev:
http://cr.openjdk.java.net/~xuelei/8129988/webrev.04/
On 1/4/2017 1:27 PM, Sean Mullan wrote:
It's unfortunate that the previous code read the system properties each
time a TrustManagerFactory was initialized. I think in practice, if
applications are changing these system properties
It's unfortunate that the previous code read the system properties each
time a TrustManagerFactory was initialized. I think in practice, if
applications are changing these system properties on the fly it isn't
going to work predictably. This implementation would have been much
simpler if the pr
Updated to use synchronized method. Most of the time, the synchronized
only perform the read access to the variables. The impact on performance
should be acceptable.
http://cr.openjdk.java.net/~xuelei/8129988/webrev.03/
Only the TrustStoreManager.java implementation get updated in this web
The changes look fine. Please ensure that the tests pass on JPRT solaris
platforms before pushing.
Approved.
Regards,
Sean.
On 04/01/17 07:02, Nikita Jain wrote:
Sorry, I made a mistake. The trivial backport requires a code review. JDK8
webrev link is mentioned in previous mail.
The ch
