You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by rg...@apache.org on 2004/11/10 05:36:33 UTC

svn commit: rev 57134 - cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/thread

Author: rgoers
Date: Tue Nov  9 20:36:31 2004
New Revision: 57134

Modified:
   cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/thread/DefaultRunnableManager.java
Log:
bug 32138 - use local logger variable.


Modified: cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/thread/DefaultRunnableManager.java
==============================================================================
--- cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/thread/DefaultRunnableManager.java	(original)
+++ cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/thread/DefaultRunnableManager.java	Tue Nov  9 20:36:31 2004
@@ -819,12 +819,12 @@
          */
         void execute(  )
         {
-            if( getLogger(  ).isDebugEnabled(  ) )
+            if( m_logger.isDebugEnabled(  ) )
             {
-                getLogger(  ).debug( "Hand over Command " +
-                                     m_command.toString(  ) + " to pool \"" +
-                                     m_pool.getName(  ) + "\" with delay=" + m_delay +
-                                     " and interval=" + m_interval );
+                m_logger.debug( "Hand over Command " +
+                                 m_command.toString(  ) + " to pool \"" +
+                                 m_pool.getName(  ) + "\" with delay=" + m_delay +
+                                 " and interval=" + m_interval );
             }
 
             synchronized( m_commandStack )

Re: svn commit: rev 57134 - cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/thread

Posted by Giacomo Pati <gi...@apache.org>.
On Wed, 10 Nov 2004 rgoers@apache.org wrote:

>         void execute(  )
>         {
> -            if( getLogger(  ).isDebugEnabled(  ) )
> +            if( m_logger.isDebugEnabled(  ) )
>             {
> -                getLogger(  ).debug( "Hand over Command " +
> -                                     m_command.toString(  ) + " to pool \"" +
> -                                     m_pool.getName(  ) + "\" with delay=" + m_delay +
> -                                     " and interval=" + m_interval );
> +                m_logger.debug( "Hand over Command " +
> +                                 m_command.toString(  ) + " to pool \"" +
> +                                 m_pool.getName(  ) + "\" with delay=" + m_delay +
> +                                 " and interval=" + m_interval );
>             }
>
>             synchronized( m_commandStack )

Ouch, thanks, spotting this.

-- 
Giacomo Pati
Otego AG, Switzerland - http://www.otego.com
Orixo, the XML business alliance - http://www.orixo.com

Re: svn commit: rev 57134 - cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/thread

Posted by Giacomo Pati <gi...@apache.org>.
On Wed, 10 Nov 2004 rgoers@apache.org wrote:

>         void execute(  )
>         {
> -            if( getLogger(  ).isDebugEnabled(  ) )
> +            if( m_logger.isDebugEnabled(  ) )
>             {
> -                getLogger(  ).debug( "Hand over Command " +
> -                                     m_command.toString(  ) + " to pool \"" +
> -                                     m_pool.getName(  ) + "\" with delay=" + m_delay +
> -                                     " and interval=" + m_interval );
> +                m_logger.debug( "Hand over Command " +
> +                                 m_command.toString(  ) + " to pool \"" +
> +                                 m_pool.getName(  ) + "\" with delay=" + m_delay +
> +                                 " and interval=" + m_interval );
>             }
>
>             synchronized( m_commandStack )

Ouch, thanks, spotting this.

-- 
Giacomo Pati
Otego AG, Switzerland - http://www.otego.com
Orixo, the XML business alliance - http://www.orixo.com