AfflictionNetworks/narya
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Drive-by logging varargification.

Browse Source 
git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@5634 542714f4-19e9-0310-aa3c-eee0fc999fb1
This commit is contained in:
Ray Greenwell
2009-01-15 22:04:36 +00:00
parent 44a4981d27
commit 9d28bdbeef
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/java/com/threerings/presents/server/PresentsSession.java
+5 -5
View File
@@ -271,8 +271,8 @@ public class PresentsSession
} }
public void resolutionFailed (Name username, Exception reason) { public void resolutionFailed (Name username, Exception reason) {
log.warning("Unable to resolve new client object [oldname=" + _username + log.warning("Unable to resolve new client object",
", newname=" + username + ", reason=" + reason + "].", reason); "oldname", _username, "newname", username, "reason", reason, reason);
// let our listener know we're hosed // let our listener know we're hosed
if (ucl != null) { if (ucl != null) {
@@ -389,7 +389,7 @@ public class PresentsSession
public void resolutionFailed (Name username, Exception reason) public void resolutionFailed (Name username, Exception reason)
{ {
// urk; nothing to do but complain and get the f**k out of dodge // 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 // end the session now to prevent danglage
endSession(); endSession();
@@ -416,7 +416,7 @@ public class PresentsSession
// message that we received // message that we received
MessageDispatcher disp = _disps.get(message.getClass()); MessageDispatcher disp = _disps.get(message.getClass());
if (disp == null) { if (disp == null) {
log.warning("No dispatcher for message [msg=" + message + "]."); log.warning("No dispatcher for message", "msg", message);
return; 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 // check to see if we've already got a connection object, in which case it's probably stale
Connection oldconn = getConnection(); Connection oldconn = getConnection();
if (oldconn != null && !oldconn.isClosed()) { 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) // close the old connection (which results in everything being properly unregistered)
oldconn.close(); oldconn.close();
} }