You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@excalibur.apache.org by le...@apache.org on 2004/07/30 09:53:22 UTC

svn commit: rev 30970 - excalibur/trunk/containerkit/logger/src/java/org/apache/avalon/excalibur/logger/factory

Author: leif
Date: Fri Jul 30 00:53:22 2004
New Revision: 30970

Modified:
   excalibur/trunk/containerkit/logger/src/java/org/apache/avalon/excalibur/logger/factory/PriorityFilterTargetFactory.java
Log:
Fix a problem where wrapped filters were not being closed on shutdown.  It is now possible to configure whether or not you want the wrapped filter to be closed.  But the default is to close it.

Modified: excalibur/trunk/containerkit/logger/src/java/org/apache/avalon/excalibur/logger/factory/PriorityFilterTargetFactory.java
==============================================================================
--- excalibur/trunk/containerkit/logger/src/java/org/apache/avalon/excalibur/logger/factory/PriorityFilterTargetFactory.java	(original)
+++ excalibur/trunk/containerkit/logger/src/java/org/apache/avalon/excalibur/logger/factory/PriorityFilterTargetFactory.java	Fri Jul 30 00:53:22 2004
@@ -66,12 +66,18 @@
     {
         final String loglevel = configuration.getAttribute( "log-level", "INFO" );
         getLogger().debug( "loglevel is " + loglevel );
-        final PriorityFilteringTarget filter = new PriorityFilteringTarget( Priority.getPriorityForName( loglevel ) );
+        
+        final boolean closeWrappedTargets =
+            configuration.getAttributeAsBoolean( "close-wrapped-targets", true );
+        
+        final PriorityFilteringTarget filter = new PriorityFilteringTarget(
+            Priority.getPriorityForName( loglevel ), closeWrappedTargets );
 
         final Configuration[] configs = configuration.getChildren();
         for( int i = 0; i < configs.length; i++ )
         {
-            final LogTargetFactory factory = m_logTargetFactoryManager.getLogTargetFactory( configs[ i ].getName() );
+            final LogTargetFactory factory =
+                m_logTargetFactoryManager.getLogTargetFactory( configs[ i ].getName() );
 
             if( null == factory )
             {
@@ -79,7 +85,8 @@
                                                   + "' at " + configs[ i ].getLocation() );
             }
 
-            getLogger().debug( "creating target " + configs[ i ].getName() + ": " + configs[ i ].toString() );
+            getLogger().debug(
+                "creating target " + configs[ i ].getName() + ": " + configs[ i ].toString() );
             final LogTarget logtarget = factory.createTarget( configs[ i ] );
             filter.addTarget( logtarget );
         }

---------------------------------------------------------------------
To unsubscribe, e-mail: scm-unsubscribe@excalibur.apache.org
For additional commands, e-mail: scm-help@excalibur.apache.org