On 14/01/2010, sebb wrote:
> On 14/01/2010, Phil Steitz wrote:
> > Phil Steitz wrote:
> > > sebb wrote:
> > >> The method SharedPoolDataSource.getPooledConnectionAndInfo has the
> > >> following code:
> > >>
> > >> synchronized (userKeys) {
> > >> if (user
On 14/01/2010, Phil Steitz wrote:
> Phil Steitz wrote:
> > sebb wrote:
> >> The method SharedPoolDataSource.getPooledConnectionAndInfo has the
> >> following code:
> >>
> >> synchronized (userKeys) {
> >> if (userKeys.containsKey(username)) {
> >>
Phil Steitz wrote:
> sebb wrote:
>> The method SharedPoolDataSource.getPooledConnectionAndInfo has the
>> following code:
>>
>> synchronized (userKeys) {
>> if (userKeys.containsKey(username)) {
>> userKeys.remove(username);
>> }
>>
sebb wrote:
> The method SharedPoolDataSource.getPooledConnectionAndInfo has the
> following code:
>
> synchronized (userKeys) {
> if (userKeys.containsKey(username)) {
> userKeys.remove(username);
> }
> }
>
> Why not jus
The method SharedPoolDataSource.getPooledConnectionAndInfo has the
following code:
synchronized (userKeys) {
if (userKeys.containsKey(username)) {
userKeys.remove(username);
}
}
Why not just use userKeys.remove(username)?
