From 9d28bdbeef5a708d4358cce5558bf0f28d2b14da Mon Sep 17 00:00:00 2001
From: Ray Greenwell <ray@threerings.net>
Date: Thu, 15 Jan 2009 22:04:36 +0000
Subject: [PATCH] Drive-by logging varargification.

git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@5634 542714f4-19e9-0310-aa3c-eee0fc999fb1
---
 .../threerings/presents/server/PresentsSession.java    | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/java/com/threerings/presents/server/PresentsSession.java b/src/java/com/threerings/presents/server/PresentsSession.java
index 9d4bfb589..d7a3a94e0 100644
--- a/src/java/com/threerings/presents/server/PresentsSession.java
+++ b/src/java/com/threerings/presents/server/PresentsSession.java
@@ -271,8 +271,8 @@ public class PresentsSession
             }
 
             public void resolutionFailed (Name username, Exception reason) {
-                log.warning("Unable to resolve new client object [oldname=" + _username +
-                        ", newname=" + username + ", reason=" + reason + "].", reason);
+                log.warning("Unable to resolve new client object",
+                    "oldname", _username, "newname", username, "reason", reason, reason);
 
                 // let our listener know we're hosed
                 if (ucl != null) {
@@ -389,7 +389,7 @@ public class PresentsSession
     public void resolutionFailed (Name username, Exception reason)
     {
         // urk; nothing to do but complain and get the f**k out of dodge
-        log.warning("Unable to resolve client [username=" + username + "].", reason);
+        log.warning("Unable to resolve client", "username", username, reason);
 
         // end the session now to prevent danglage
         endSession();
@@ -416,7 +416,7 @@ public class PresentsSession
         // message that we received
         MessageDispatcher disp = _disps.get(message.getClass());
         if (disp == null) {
-            log.warning("No dispatcher for message [msg=" + message + "].");
+            log.warning("No dispatcher for message", "msg", message);
             return;
         }
 
@@ -482,7 +482,7 @@ public class PresentsSession
         // check to see if we've already got a connection object, in which case it's probably stale
         Connection oldconn = getConnection();
         if (oldconn != null && !oldconn.isClosed()) {
-            log.info("Closing stale connection [old=" + oldconn + ", new=" + conn + "].");
+            log.info("Closing stale connection", "old", oldconn, "new", conn);
             // close the old connection (which results in everything being properly unregistered)
             oldconn.close();
         }