Hi Claes, What's the necessary to get all system properties for just one specific one? Can you explain more about the necessary to make the change?
Thanks, Xuelei On 6/8/2016 3:44 AM, Claes Redestad wrote: > 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 >>
