Prettier string representation of ChannelEvent implementations

src/main/java/org/jboss/netty/channel/DefaultChildChannelStateEvent.java

@@ -69,10 +69,10 @@ public final class DefaultChildChannelStateEvent implements ChildChannelStateEve
         String channelString = getChannel().toString();
         StringBuilder buf = new StringBuilder(channelString.length() + 32);
         buf.append(channelString);
-        buf.append(" - (childId: ");
+        buf.append(" - (");
-        buf.append(getChildChannel().getId().toString());
+        buf.append(getChildChannel().isOpen()? "CHILD_OPEN" : "CHILD_CLOSED");
-        buf.append(", childState: ");
+        buf.append(": ");
-        buf.append(getChildChannel().isOpen()? "OPEN" : "CLOSED");
+        buf.append(getChildChannel().getId());
         buf.append(')');
         return buf.toString();
     }

src/main/java/org/jboss/netty/channel/DefaultExceptionEvent.java

@@ -69,6 +69,6 @@ public final class DefaultExceptionEvent implements ExceptionEvent {
 
     @Override
     public String toString() {
-        return getChannel().toString() + " - (cause: " + cause.getClass().getSimpleName() + ')';
+        return getChannel().toString() + " - (EXCEPTION: " + cause + ')';
     }
 }

src/main/java/org/jboss/netty/channel/DefaultWriteCompletionEvent.java

@@ -67,12 +67,24 @@ public final class DefaultWriteCompletionEvent implements WriteCompletionEvent {
 
     @Override
     public String toString() {
-        String parentString = super.toString();
+        String channelString = getChannel().toString();
-        StringBuilder buf = new StringBuilder(parentString.length() + 32);
+        StringBuilder buf = new StringBuilder(channelString.length() + 32);
-        buf.append(parentString);
+        buf.append(channelString);
-        buf.append(" - (writtenAmount: ");
+        buf.append(" - (WROTE: ");
-        buf.append(getWrittenAmount());
+
-        buf.append(')');
+        int amount = getWrittenAmount();
+        switch (amount) {
+        case 0:
+            buf.append("0 byte");
+            break;
+        case 1:
+            buf.append("1 byte");
+            break;
+        default:
+            buf.append(amount);
+            buf.append(" bytes");
+        }
+        buf.append(")");
         return buf.toString();
     }
 }

src/main/java/org/jboss/netty/channel/DownstreamChannelStateEvent.java

@@ -81,7 +81,7 @@ public final class DownstreamChannelStateEvent implements ChannelStateEvent {
         String channelString = getChannel().toString();
         StringBuilder buf = new StringBuilder(channelString.length() + 64);
         buf.append(channelString);
-        buf.append(" - (request: ");
+        buf.append(" - (");
         switch (getState()) {
         case OPEN:
             if (Boolean.TRUE.equals(getValue())) {
@@ -92,20 +92,23 @@ public final class DownstreamChannelStateEvent implements ChannelStateEvent {
             break;
         case BOUND:
             if (getValue() != null) {
-                buf.append("BIND");
+                buf.append("BIND: ");
+                buf.append(getValue());
             } else {
                 buf.append("UNBIND");
             }
             break;
         case CONNECTED:
             if (getValue() != null) {
-                buf.append("CONNECT");
+                buf.append("CONNECT: ");
+                buf.append(getValue());
             } else {
                 buf.append("DISCONNECT");
             }
             break;
         case INTEREST_OPS:
-            buf.append("CHANGE_INTEREST");
+            buf.append("CHANGE_INTEREST: ");
+            buf.append(getValue());
             break;
         default:
             buf.append(getState().name());

src/main/java/org/jboss/netty/channel/DownstreamMessageEvent.java

@@ -85,9 +85,9 @@ public final class DownstreamMessageEvent implements MessageEvent {
     @Override
     public String toString() {
         if (remoteAddress == null) {
-            return super.toString() + " - (write: " + message + ')';
+            return getChannel().toString() + " - (WRITE: " + message + ')';
         } else {
-            return super.toString() + " - (write: " + message + ", " + remoteAddress + ')';
+            return getChannel().toString() + " - (WRITE: " + message + ", " + remoteAddress + ')';
         }
     }
 }

src/main/java/org/jboss/netty/channel/UpstreamChannelStateEvent.java

@@ -78,7 +78,7 @@ public final class UpstreamChannelStateEvent implements ChannelStateEvent {
         String channelString = getChannel().toString();
         StringBuilder buf = new StringBuilder(channelString.length() + 64);
         buf.append(channelString);
-        buf.append(" - (state: ");
+        buf.append(" - (");
         switch (getState()) {
         case OPEN:
             if (Boolean.TRUE.equals(getValue())) {
@@ -89,14 +89,16 @@ public final class UpstreamChannelStateEvent implements ChannelStateEvent {
             break;
         case BOUND:
             if (getValue() != null) {
-                buf.append("BOUND");
+                buf.append("BOUND: ");
+                buf.append(getValue());
             } else {
                 buf.append("UNBOUND");
             }
             break;
         case CONNECTED:
             if (getValue() != null) {
-                buf.append("CONNECTED");
+                buf.append("CONNECTED: ");
+                buf.append(getValue());
             } else {
                 buf.append("DISCONNECTED");
             }

src/main/java/org/jboss/netty/channel/UpstreamMessageEvent.java

@@ -81,9 +81,9 @@ public final class UpstreamMessageEvent implements MessageEvent {
     @Override
     public String toString() {
         if (remoteAddress == null) {
-            return super.toString() + " - (received: " + message + ')';
+            return getChannel().toString() + " - (RECEIVED: " + message + ')';
         } else {
-            return super.toString() + " - (received: " + message + ", " + remoteAddress + ')';
+            return getChannel().toString() + " - (RECEIVED: " + message + ", " + remoteAddress + ')';
         }
     }
 }