Looking for a reviewer for this trivial change to clarify the long outstanding 
behavior of registererDriver:


$ hg diff DriverManager.java 
diff -r 262a625809fd src/share/classes/java/sql/DriverManager.java
--- a/src/share/classes/java/sql/DriverManager.java     Thu Sep 12 01:47:05 
2013 -0700
+++ b/src/share/classes/java/sql/DriverManager.java     Thu Sep 12 10:35:48 
2013 -0400
@@ -326,6 +326,7 @@
      * @param driver the new JDBC Driver that is to be registered with the
      *               {@code DriverManager}
      * @exception SQLException if a database access error occurs
+     * @exception NullPointerException if {@code driver} is null
      */
     public static synchronized void registerDriver(java.sql.Driver driver)
         throws SQLException {
@@ -345,6 +346,7 @@
      * @param da     the {@code DriverAction} implementation to be used when
      *               {@code DriverManager#deregisterDriver} is called
      * @exception SQLException if a database access error occurs
+     * @exception NullPointerException if {@code driver} is null
      */
     public static synchronized void registerDriver(java.sql.Driver driver,
             DriverAction da)


registerDriver has thrown this NPE since the early days of DriverManager (1997) 
so it was requested that we clarify this.

Alan,  could you please confirm whether I need a CCC given this behavior has 
been there since JDBC 1.0 and this just adds the @exception and there is no 
behavior change.

Best
Lance

Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com

Reply via email to