Annette Doyle wrote:

> In the constructor of class
> org.apache.avalon.excalibur.datasource.JdbcConnection, there is a bug in the
> following if statement:
> 


<snip/>


> The condition in the if statement should be the following:
> 
> if (null != keepAlive && !("".equals(keepAlive.trim())))
> 
> There should be a boolean NOT ! operator before "".equals(keepAlive.trim()).


Thank you very much for catching that.  We are fallible beings after all....

I have fixed CVS.


-- 

"They that give up essential liberty to obtain a little temporary safety
  deserve neither liberty nor safety."
                 - Benjamin Franklin


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

Reply via email to