From 15aded0df78cb63317d622c302214ed76905e7b0 Mon Sep 17 00:00:00 2001 From: Trustin Lee Date: Thu, 17 Dec 2009 10:21:34 +0000 Subject: [PATCH] * More documentation for developers --- .../java/org/jboss/netty/handler/timeout/IdleStateHandler.java | 3 +++ .../org/jboss/netty/handler/timeout/ReadTimeoutHandler.java | 3 +++ 2 files changed, 6 insertions(+) diff --git a/src/main/java/org/jboss/netty/handler/timeout/IdleStateHandler.java b/src/main/java/org/jboss/netty/handler/timeout/IdleStateHandler.java index 0d87b6da8e..647b359e67 100644 --- a/src/main/java/org/jboss/netty/handler/timeout/IdleStateHandler.java +++ b/src/main/java/org/jboss/netty/handler/timeout/IdleStateHandler.java @@ -206,6 +206,9 @@ public class IdleStateHandler extends SimpleChannelUpstreamHandler // this.channelOpen() will not be invoked. // We have to initialize here instead. initialize(ctx); + } else { + // channelOpen event has not been fired yet. + // this.channelOpen() will be invoked and initialization will occur there. } } diff --git a/src/main/java/org/jboss/netty/handler/timeout/ReadTimeoutHandler.java b/src/main/java/org/jboss/netty/handler/timeout/ReadTimeoutHandler.java index da3a996b94..8604fbd7a7 100644 --- a/src/main/java/org/jboss/netty/handler/timeout/ReadTimeoutHandler.java +++ b/src/main/java/org/jboss/netty/handler/timeout/ReadTimeoutHandler.java @@ -125,6 +125,9 @@ public class ReadTimeoutHandler extends SimpleChannelUpstreamHandler // this.channelOpen() will not be invoked. // We have to initialize here instead. initialize(ctx); + } else { + // channelOpen event has not been fired yet. + // this.channelOpen() will be invoked and initialization will occur there. } }