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

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

Author: david.lloyd at jboss.com
Date: 2009-05-20 17:47:45 -0400 (Wed, 20 May 2009)
New Revision: 3187

Modified:
   jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java
Log:
Minor optimization

Modified: jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java
===================================================================
--- jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java	2009-05-20 21:44:59 UTC (rev 3186)
+++ jboss-logmanager/trunk/src/main/java/org/jboss/logmanager/Logger.java	2009-05-20 21:47:45 UTC (rev 3187)
@@ -269,8 +269,16 @@
      * @param record the log record
      */
     private void doLog(final LogRecord record) {
-        final ExtLogRecord extRecord = (record instanceof ExtLogRecord) ? (ExtLogRecord) record : new ExtLogRecord(record, LOGGER_CLASS_NAME);
-        extRecord.setLoggerName(getName());
+        doLog((record instanceof ExtLogRecord) ? (ExtLogRecord) record : new ExtLogRecord(record, LOGGER_CLASS_NAME));
+    }
+
+    /**
+     * Do the logging with no level checks (they've already been done).
+     *
+     * @param record the log record
+     */
+    private void doLog(final ExtLogRecord record) {
+        record.setLoggerName(getName());
         String bundleName = null;
         ResourceBundle bundle = null;
         for (Logger current = this; current != null; current = current.getParent()) {
@@ -281,12 +289,12 @@
             }
         }
         if (bundleName != null && bundle != null) {
-            extRecord.setResourceBundleName(bundleName);
-            extRecord.setResourceBundle(bundle);
+            record.setResourceBundleName(bundleName);
+            record.setResourceBundle(bundle);
         }
         final Filter filter = this.filter;
         try {
-            if (filter != null && ! filter.isLoggable(extRecord)) {
+            if (filter != null && ! filter.isLoggable(record)) {
                 return;
             }
         } catch (VirtualMachineError e) {
@@ -299,7 +307,7 @@
             final Handler[] handlers = current.handlers;
             if (handlers != null) {
                 for (Handler handler : handlers) try {
-                    handler.publish(extRecord);
+                    handler.publish(record);
                 } catch (VirtualMachineError e) {
                     throw e;
                 } catch (Throwable t) {