geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xuhaih...@apache.org
Subject svn commit: r1171452 - /geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/GeronimoFactoryFinderProvider.java
Date Fri, 16 Sep 2011 08:52:03 GMT
Author: xuhaihong
Date: Fri Sep 16 08:52:03 2011
New Revision: 1171452

URL: http://svn.apache.org/viewvc?rev=1171452&view=rev
Log:
Minor change to the debug log output, make sure the factory map could be print before it is
cleared

Modified:
    geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/GeronimoFactoryFinderProvider.java

Modified: geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/GeronimoFactoryFinderProvider.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/GeronimoFactoryFinderProvider.java?rev=1171452&r1=1171451&r2=1171452&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/GeronimoFactoryFinderProvider.java
(original)
+++ geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/GeronimoFactoryFinderProvider.java
Fri Sep 16 08:52:03 2011
@@ -304,13 +304,13 @@ public class GeronimoFactoryFinderProvid
             // _registeredFactoryNames has as value type Map<String,List> and this
must
             // be cleaned before release (for gc).
             Map<String, List<String>> factoryClassNames = _registeredFactoryNames.get(webApplicationIdentity);
+            if(logger.isDebugEnabled()) {
+                logger.debug("Web application [" + webApplicationIdentity + "] releases the
factory map " + factoryClassNames);
+            }
             if (factoryClassNames != null) {
                 factoryClassNames.clear();
             }
-            Map<String, List<String>>factoryMap = _registeredFactoryNames.remove(webApplicationIdentity);
-            if(logger.isDebugEnabled()) {
-                logger.debug("Web application [" + webApplicationIdentity + "] releases the
factory map " + factoryMap);
-            }
+            _registeredFactoryNames.remove(webApplicationIdentity);            
         }
     }
 



Mime
View raw message