User: ejort   
  Date: 02/01/02 09:06:54

  Modified:    src/main/org/jboss/mq/server MessageCache.java
  Log:
  Removed some unguarded trace statements.
  
  Revision  Changes    Path
  1.11      +17 -9     jbossmq/src/main/org/jboss/mq/server/MessageCache.java
  
  Index: MessageCache.java
  ===================================================================
  RCS file: /cvsroot/jboss/jbossmq/src/main/org/jboss/mq/server/MessageCache.java,v
  retrieving revision 1.10
  retrieving revision 1.11
  diff -u -r1.10 -r1.11
  --- MessageCache.java 2001/12/13 22:22:43     1.10
  +++ MessageCache.java 2002/01/02 17:06:53     1.11
  @@ -25,7 +25,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Hiram Chirino</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>David Maplesden</a>
  - * @version    $Revision: 1.10 $
  + * @version    $Revision: 1.11 $
    */
   public class MessageCache extends ServiceMBeanSupport implements MessageCacheMBean, 
MBeanRegistration, Runnable
   {
  @@ -63,7 +63,8 @@
       */
      public MessageReference add(SpyMessage message) throws javax.jms.JMSException
      {
  -      log.trace("add lock aquire");
  +      if (log.isTraceEnabled())
  +        log.trace("add lock aquire");
         synchronized (this)
         {
            MessageReference mh = new MessageReference();
  @@ -72,7 +73,8 @@
            totalCacheSize++;
            validateSoftReferenceDepth();
   
  -         log.trace("add lock release");
  +         if (log.isTraceEnabled())
  +           log.trace("add lock release");
            return mh;
         }
      }
  @@ -82,13 +84,15 @@
       */
      public void remove(MessageReference mr) throws JMSException
      {
  -      log.trace("remove lock aquire");
  +      if (log.isTraceEnabled())
  +        log.trace("remove lock aquire");
         synchronized (this)
         {
            mr.clear();
            lruCache.remove(mr);
            totalCacheSize--;
  -         log.trace("remove lock release");
  +         if (log.isTraceEnabled())
  +           log.trace("remove lock release");
         }
      }
   
  @@ -137,7 +141,8 @@
         {
            // Signal to exit the thread.
         }
  -      log.debug("Thread exiting.");
  +      if (log.isDebugEnabled())
  +        log.debug("Thread exiting.");
      }
   
      /**
  @@ -170,7 +175,8 @@
            // if the serverity is low since it will round up.
            if (chnageCount > 1)
            {
  -            log.debug("Converting " + chnageCount + " hard ref to to soft refs");
  +            if (log.isDebugEnabled())
  +              log.debug("Converting " + chnageCount + " hard ref to to soft refs");
               Node leastRecent = lruCache.getLeastRecent();
               for (int i = 0; i < chnageCount && leastRecent != null; i++)
               {
  @@ -191,14 +197,16 @@
       */
      synchronized public void messageReferenceUsedEvent(MessageReference mh, boolean 
wasHard)
      {
  -      log.trace("messageReferenceUsedEvent lock aquire");
  +      if (log.isTraceEnabled())
  +        log.trace("messageReferenceUsedEvent lock aquire");
         synchronized (this)
         {
            if (wasHard)
               lruCache.makeMostRecent(mh);
            else
               lruCache.addMostRecent(mh);
  -         log.trace("messageReferenceUsedEvent lock released");
  +         if (log.isTraceEnabled())
  +           log.trace("messageReferenceUsedEvent lock released");
         }
      }
   
  
  
  

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

Reply via email to