svn commit: r580464 - /mina/trunk/core/src/main/java/org/apache/mina/common/IoSessionLogger.java

[maarten]

Author: maarten
Date: Fri Sep 28 12:21:10 2007
New Revision: 580464

URL: http://svn.apache.org/viewvc?rev=580464&view=rev
Log:
Resolved issue: DIRMINA-445 : IoSessionLogger methods now take an Object 
instead of String (avoiding log message composition when logging event is 
filtered out)

Modified:
    mina/trunk/core/src/main/java/org/apache/mina/common/IoSessionLogger.java

Modified: 
mina/trunk/core/src/main/java/org/apache/mina/common/IoSessionLogger.java
URL: 
http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/common/IoSessionLogger.java?rev=580464&r1=580463&r2=580464&view=diff
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/common/IoSessionLogger.java 
(original)
+++ mina/trunk/core/src/main/java/org/apache/mina/common/IoSessionLogger.java 
Fri Sep 28 12:21:10 2007
@@ -49,13 +49,13 @@
         return session.getHandler().getClass();
     }
 
-    public static void trace(IoSession session, String message) {
+    public static void trace(IoSession session, Object message) {
         trace(getLogger(session), session, message);
     }
 
-    public static void trace(Logger log, IoSession session, String message) {
+    public static void trace(Logger log, IoSession session, Object message) {
         if (log.isTraceEnabled()) {
-            log.trace(String.valueOf(getPrefix(session)) + message);
+            log.trace("{}{}", getPrefix(session), message);
         }
     }
 
@@ -66,17 +66,17 @@
     public static void trace(Logger log, IoSession session, String message,
             Throwable cause) {
         if (log.isTraceEnabled()) {
-            log.trace(String.valueOf(getPrefix(session)) + message, cause);
+            log.trace(getPrefix(session) + message, cause);
         }
     }
 
-    public static void debug(IoSession session, String message) {
+    public static void debug(IoSession session, Object message) {
         debug(getLogger(session), session, message);
     }
 
-    public static void debug(Logger log, IoSession session, String message) {
+    public static void debug(Logger log, IoSession session, Object message) {
         if (log.isDebugEnabled()) {
-            log.debug(String.valueOf(getPrefix(session)) + message);
+            log.debug("{}{}", getPrefix(session), message);
         }
     }
 
@@ -87,17 +87,17 @@
     public static void debug(Logger log, IoSession session, String message,
             Throwable cause) {
         if (log.isDebugEnabled()) {
-            log.debug(String.valueOf(getPrefix(session)) + message, cause);
+            log.debug(getPrefix(session) + message, cause);
         }
     }
 
-    public static void info(IoSession session, String message) {
+    public static void info(IoSession session, Object message) {
         info(getLogger(session), session, message);
     }
 
-    public static void info(Logger log, IoSession session, String message) {
+    public static void info(Logger log, IoSession session, Object message) {
         if (log.isInfoEnabled()) {
-            log.info(String.valueOf(getPrefix(session)) + message);
+            log.info("{}{}", getPrefix(session), message);
         }
     }
 
@@ -108,17 +108,17 @@
     public static void info(Logger log, IoSession session, String message,
             Throwable cause) {
         if (log.isInfoEnabled()) {
-            log.info(String.valueOf(getPrefix(session)) + message, cause);
+            log.info(getPrefix(session) + message, cause);
         }
     }
 
-    public static void warn(IoSession session, String message) {
+    public static void warn(IoSession session, Object message) {
         warn(getLogger(session), session, message);
     }
 
-    public static void warn(Logger log, IoSession session, String message) {
+    public static void warn(Logger log, IoSession session, Object message) {
         if (log.isWarnEnabled()) {
-            log.warn(String.valueOf(getPrefix(session)) + message);
+            log.warn("{}{}", getPrefix(session), message);
         }
     }
 
@@ -133,17 +133,17 @@
     public static void warn(Logger log, IoSession session, String message,
             Throwable cause) {
         if (log.isWarnEnabled()) {
-            log.warn(String.valueOf(getPrefix(session)) + message, cause);
+            log.warn(getPrefix(session) + message, cause);
         }
     }
 
-    public static void error(IoSession session, String message) {
+    public static void error(IoSession session, Object message) {
         error(getLogger(session), session, message);
     }
 
-    public static void error(Logger log, IoSession session, String message) {
+    public static void error(Logger log, IoSession session, Object message) {
         if (log.isErrorEnabled()) {
-            log.error(String.valueOf(getPrefix(session)) + message);
+            log.error("{}{}", getPrefix(session), message);
         }
     }
 
@@ -158,7 +158,7 @@
     public static void error(Logger log, IoSession session, String message,
             Throwable cause) {
         if (log.isErrorEnabled()) {
-            log.error(String.valueOf(getPrefix(session)) + message, cause);
+            log.error(getPrefix(session) + message, cause);
         }
     }