[jboss-cvs] JBoss Messaging SVN: r3832 - trunk/src/main/org/jboss/messaging/core/remoting/impl/mina.

[jboss-cvs-commits at lists.jboss.org]

Author: jmesnil
Date: 2008-02-29 05:34:48 -0500 (Fri, 29 Feb 2008)
New Revision: 3832

Modified:
   trunk/src/main/org/jboss/messaging/core/remoting/impl/mina/MinaHandler.java
Log:
* check trace level is enabled before logging

Modified: trunk/src/main/org/jboss/messaging/core/remoting/impl/mina/MinaHandler.java
===================================================================
--- trunk/src/main/org/jboss/messaging/core/remoting/impl/mina/MinaHandler.java	2008-02-29 10:34:38 UTC (rev 3831)
+++ trunk/src/main/org/jboss/messaging/core/remoting/impl/mina/MinaHandler.java	2008-02-29 10:34:48 UTC (rev 3832)
@@ -78,7 +78,8 @@
    {
       if (message instanceof Response)
       {
-         log.trace("received response " + message);
+         if (log.isTraceEnabled())
+            log.trace("received response " + message);
          // response is handled by the reqres filter.
          // do nothing
          return;
@@ -86,7 +87,8 @@
       
       if (message instanceof Ping)
       {
-         log.trace("received ping " + message);
+         if (log.isTraceEnabled())
+            log.trace("received ping " + message);
          // response is handled by the keep-alive filter.
          // do nothing
          return;