Hi,
there is some lingering concern that this and related changes make it
that much easier to accidentally leak the system Properties object
outside of core modules. By wrapping access to the system Properties
object in a class residing in a non-exported package we disallow this at
little to no cost:
http://cr.openjdk.java.net/~redestad/8155039/webrev.02/
/Claes
On 2016-05-12 15:37, Claes Redestad wrote:
Hi,
the API this improvement depends on was updated to reflect more clearly
that this
is taking a privileged action:
https://bugs.openjdk.java.net/browse/JDK-8155775
Here's the updated webrev:
http://cr.openjdk.java.net/~redestad/8155039/webrev.01/
Thanks!
/Claes
On 2016-04-25 19:28, Claes Redestad wrote:
Hi,
SSLContextImpl and TrustManagerFactoryImpl has some setup code that
could be
simplified, getting rid of a couple of anonymous classes in the process.
Webrev: http://cr.openjdk.java.net/~redestad/8155039/webrev.00
Bug: https://bugs.openjdk.java.net/browse/JDK-8155039
Alternatively we could remove OpenFileInputStreamAction instead since
these two uses
introduced here are actually the only active uses of this old
convenience class.
Thanks!
/Claes
