RFR 824767: Incorrect class name displayed in DriverManager trace output

[Joe Wang]

+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.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>
>
>