[Jboss-cvs] JBoss Messaging SVN: r1248 - branches/Branch_1_0/src/main/org/jboss/jms/server/endpoint

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

Author: ovidiu.feodorov at jboss.com
Date: 2006-08-31 22:32:34 -0400 (Thu, 31 Aug 2006)
New Revision: 1248

Modified:
   branches/Branch_1_0/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java
Log:
gave a little more flexibility in cuting out the performance overhead of the 'anti-dummy' log statement for systems that choose to go with INFO or higher threshold

Modified: branches/Branch_1_0/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java
===================================================================
--- branches/Branch_1_0/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java	2006-09-01 01:20:24 UTC (rev 1247)
+++ branches/Branch_1_0/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java	2006-09-01 02:32:34 UTC (rev 1248)
@@ -224,9 +224,15 @@
          // channel for delivery later.
          if (!started)
          {
-            // this is a common programming error, make this visible in the debug logs
-            // TODO: analyse performance implications
-            log.debug(this + " NOT started yet!");
+            // this is a common programming error, make this visible in the debug logs. However,
+            // make also possible to cut out the performance overhead for systems that raise the
+            // threshold to INFO or higher.
+
+            if (log.isDebugEnabled())
+            {
+               log.debug(this + " NOT started yet!");
+            }
+
             return null;
          }