You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xu...@apache.org on 2011/09/16 10:52:03 UTC
svn commit: r1171452 -
/geronimo/server/trunk/plugins/myfaces/geronimo-myfaces/src/main/java/org/apache/geronimo/myfaces/GeronimoFactoryFinderProvider.java
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);
}
}