sebb wrote: > On 05/01/2010, Phil Steitz <[email protected]> wrote: >> sebb wrote: >> > It looks like the multi-threaded tests in TestSharedPoolDataSource >> > have the same problems as the ones in TestPerUserPoolDataSource. >> > >> > I'm currently testing to see if I can trigger a failure. >> > >> > I also noticed that there are quite a few instances of "data hiding" - >> > i.e. using the same name for a local variable as another variable in >> > scope. I've fixed all such errors in test code - is it OK to do the >> > same for the release code? >> >> >> Yes, but please open JIRAs and update the changelog for these changes. > > OK, DBCP-314 opened, fixed and resolved. > > Hope that's OK.
Looks good. Thanks! Phil > >> Phil >> > >> > --------------------------------------------------------------------- >> > To unsubscribe, e-mail: [email protected] >> > For additional commands, e-mail: [email protected] >> > >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
