>               //Shouldn't ever happen, but it did, just return null.  <---
remove this
>  +            // Exception from DriverManager.getConnection() - log it,
and return null
>  +            StringWriter sout = new StringWriter();
>  +            PrintWriter pout = new PrintWriter(sout, true);
>  +            pout.println("Error creating connection: ");
>  +            sqle.printStackTrace(pout);
>  +            getLogger().error(sout.toString());
>  +             return null;    <-- remove this
>               return null;

Somehow this ended up with two return statements (and an extra comment --
the line removed in my patch didn't get removed).

        --- Noel


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to