RFR 8060068 : Remove the static initializer block from DriverManager
Mandy Chung
mandy.chung at oracle.com
Wed Dec 3 04:28:46 UTC 2014
On 12/2/2014 1:47 PM, Lance Andersen wrote:
> The revised webrev is here
> http://cr.openjdk.java.net/~lancea/8060068/webrev.03/
> <http://cr.openjdk.java.net/%7Elancea/8060068/webrev.03/>
>
Looks good. Nit: line 443 and a few places in the getConnection need a
space in "for(", "if("
> Note, I also tweaked the doPriviliged block for the JDBC property
It's nice to see use of limited doPrivileged. Limited doPrivileged
restricts the permissions be accessed by the doPrivileged block. On the
other hand, since it only calls System.getProperty, that won't leak any
privileges to untrusted code. I think we would need some guideline what
can benefit from limited doPrivileged. Anyway, I'm fine with your change.
Mandy
More information about the core-libs-dev
mailing list