You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by da...@apache.org on 2005/05/05 04:48:51 UTC
svn commit: r168245 -
/geronimo/trunk/modules/kernel/src/java/org/apache/geronimo/kernel/log/GeronimoLogFactory.java
Author: dain
Date: Wed May 4 19:48:50 2005
New Revision: 168245
URL: http://svn.apache.org/viewcvs?rev=168245&view=rev
Log:
Applied GERONIMO-637
Modified:
geronimo/trunk/modules/kernel/src/java/org/apache/geronimo/kernel/log/GeronimoLogFactory.java
Modified: geronimo/trunk/modules/kernel/src/java/org/apache/geronimo/kernel/log/GeronimoLogFactory.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/kernel/src/java/org/apache/geronimo/kernel/log/GeronimoLogFactory.java?rev=168245&r1=168244&r2=168245&view=diff
==============================================================================
--- geronimo/trunk/modules/kernel/src/java/org/apache/geronimo/kernel/log/GeronimoLogFactory.java (original)
+++ geronimo/trunk/modules/kernel/src/java/org/apache/geronimo/kernel/log/GeronimoLogFactory.java Wed May 4 19:48:50 2005
@@ -98,14 +98,15 @@
public void release() {
synchronized (factoryLock) {
- for (Iterator maps = instancesByClassLoader.values().iterator(); maps.hasNext();) {
- Map instances = (Map) maps.next();
- for (Iterator logs = instances.values().iterator(); logs.hasNext();) {
- GeronimoLog log = (GeronimoLog) logs.next();
- log.setLog(null);
-
- }
- }
+// TODO rethink this - it works for now
+// for (Iterator maps = instancesByClassLoader.values().iterator(); maps.hasNext();) {
+// Map instances = (Map) maps.next();
+// for (Iterator logs = instances.values().iterator(); logs.hasNext();) {
+// GeronimoLog log = (GeronimoLog) logs.next();
+// log.setLog(null);
+//
+// }
+// }
instancesByClassLoader.clear();
}
}