+1

Joe

On 7/14/20 8:41 AM, Lance Andersen wrote:
Hi all,

Please review this trivial fix for https://bugs.openjdk.java.net/browse/JDK-8247677 
<https://bugs.openjdk.java.net/browse/JDK-8247677>,  which addresses a couple 
of typos where the driver class name was not included in the trace output from 
DriverManager.

———————
$ hg diff
diff -r e92a076bc6a5 src/java.sql/share/classes/java/sql/DriverManager.java
--- a/src/java.sql/share/classes/java/sql/DriverManager.java    Fri Jun 26 
13:37:43 2020 -0400
+++ b/src/java.sql/share/classes/java/sql/DriverManager.java    Tue Jul 14 
10:15:42 2020 -0400
@@ -1,5 +1,5 @@
  /*
- * Copyright (c) 1996, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1996, 2020, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
@@ -449,7 +449,7 @@
              if (isDriverAllowed(aDriver.driver, callerClass)) {
                  result.add(aDriver.driver);
              } else {
-                println("    skipping: " + aDriver.getClass().getName());
+                println("    skipping: " + 
aDriver.driver.getClass().getName());
              }
          }
          return result;
@@ -687,7 +687,7 @@
                  }
} else {
-                println("    skipping: " + aDriver.getClass().getName());
+                println("    skipping: " + 
aDriver.driver.getClass().getName());
              }
}

——————————

Best,
Lance
  <http://oracle.com/us/design/oracle-email-sig-198324.gif>
  <http://oracle.com/us/design/oracle-email-sig-198324.gif> 
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
  <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| 
Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>




Reply via email to