[jboss-svn-commits] JBoss Common SVN: r4831 - jboss-logmanager/trunk/src/main/java/org/jboss/logmanager.

[jboss-svn-commits at lists.jboss.org]

Author: david.lloyd at jboss.com
Date: 2010-08-13 14:51:26 -0400 (Fri, 13 Aug 2010)
New Revision: 4831

Modified:
   jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java
   jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java
Log:
LOGMGR-23: fix finalization of lgger node

Modified: jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java
===================================================================
--- jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java	2010-08-13 16:51:35 UTC (rev 4830)
+++ jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java	2010-08-13 18:51:26 UTC (rev 4831)
@@ -573,22 +573,6 @@
         logRaw(rec);
     }
 
-    // GC
-
-    /**
-     * Perform finalization actions.  This amounts to clearing out the loglevel so that all children are updated
-     * with the parent's effective loglevel.  As such, a lock is acquired from this method which might cause delays in
-     * garbage collection.
-     */
-    protected void finalize() throws Throwable {
-        try {
-            // clear out level so that it spams out to all children
-            setLevel(null);
-        } finally {
-            super.finalize();
-        }
-    }
-
     // alternate SPI hooks
 
     /**

Modified: jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java
===================================================================
--- jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java	2010-08-13 16:51:35 UTC (rev 4830)
+++ jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/LoggerNode.java	2010-08-13 18:51:26 UTC (rev 4831)
@@ -431,4 +431,20 @@
     LoggerNode getParent() {
         return parent;
     }
+
+    // GC
+
+    /**
+     * Perform finalization actions.  This amounts to clearing out the loglevel so that all children are updated
+     * with the parent's effective loglevel.  As such, a lock is acquired from this method which might cause delays in
+     * garbage collection.
+     */
+    protected void finalize() throws Throwable {
+        try {
+            // clear out level so that it spams out to all children
+            setLevel(null);
+        } finally {
+            super.finalize();
+        }
+    }
 }