Sorry,

I was merging a change I made to 2.4 and didn't realize the log method had
changed.

Also, I noticed that you changed formatting. Does JBoss enforce coding
standards in
regards to formatting? If so,  where can I find this information?

Thanks,
d.

-----Original Message-----
From: Scott M Stark [mailto:[EMAIL PROTECTED]]
Sent: Wednesday, November 21, 2001 1:41 PM
To: [EMAIL PROTECTED]
Subject: [JBoss-dev] CVS update:
jbosscx/src/main/org/jboss/resource/connectionmanager
ManagedConnectionPoolFactory.java


  User: starksm 
  Date: 01/11/21 11:41:16

  Modified:    src/main/org/jboss/resource/connectionmanager
                        ManagedConnectionPoolFactory.java
  Log:
  Fix the bad check-in by thedug
  
  Revision  Changes    Path
  1.4       +9 -5
jbosscx/src/main/org/jboss/resource/connectionmanager/ManagedConnectionPoolF
actory.java
  
  Index: ManagedConnectionPoolFactory.java
  ===================================================================
  RCS file:
/cvsroot/jboss/jbosscx/src/main/org/jboss/resource/connectionmanager/Managed
ConnectionPoolFactory.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- ManagedConnectionPoolFactory.java 2001/11/21 18:14:51     1.3
  +++ ManagedConnectionPoolFactory.java 2001/11/21 19:41:16     1.4
  @@ -68,14 +68,18 @@
        * object will not be returned to the pool - after this, it is  gone.
This
        * is called when the pool shrinks, and when the pool is shut  down.
        */
  -    public void deleteObject(Object pooledObject) {
  -        try{
  +    public void deleteObject(Object pooledObject)
  +    {
  +        try
  +        {
               ((ManagedConnection)pooledObject).destroy();
  -        }catch(ResourceException e) {
  -            log.println("Unable to destroy ManagedConnection: "+e);
           }
  +        catch(ResourceException e)
  +        {
  +            log.error("Unable to destroy ManagedConnection: ", e);
  +        }
   
  -     }
  +   }
   
   
   
  
  
  

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to