Generally looks fine, but I recommend some adjustments:

I'd like to see some "()" on this line!

    470         ClassLoader cl = cc != null ? cc.getClassLoader() : null;

I prefer to update the copyright year as part of making the code changes.

-Joe

On 6/7/2012 11:47 AM, Lance Andersen - Oracle wrote:
Hi all,

Another review request (to go with the others that are outstanding).

This is change is for jigsaw, where we are removing the native code used by 
DriverManager.  The webrev can be found at:

http://cr.openjdk.java.net/~lancea/7172551/webrev.00

Best,
Lance

Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
[email protected]

Reply via email to