diff --git a/src/main/java/org/jboss/netty/channel/Channels.java b/src/main/java/org/jboss/netty/channel/Channels.java index 204ead002b..04526d8ded 100644 --- a/src/main/java/org/jboss/netty/channel/Channels.java +++ b/src/main/java/org/jboss/netty/channel/Channels.java @@ -144,6 +144,8 @@ public class Channels { // event factory methods /** + * @deprecated Instantiate {@link UpstreamMessageEvent} or {@link DownstreamMessageEvent} directly instead. + * * Creates a new {@link MessageEvent} * * @param channel the channel which is associated with the event @@ -153,11 +155,14 @@ public class Channels { * ('received' when the event is sent upstream, and * 'sent' when the event is sent downstream) */ + @Deprecated public static MessageEvent messageEvent(Channel channel, ChannelFuture future, Object message) { return messageEvent(channel, future, message, null); } /** + * @deprecated Instantiate {@link UpstreamMessageEvent} or {@link DownstreamMessageEvent} directly instead. + * * Creates a new {@link MessageEvent} * * @param channel the channel which is associated with the event @@ -170,6 +175,7 @@ public class Channels { * ('source' when the event is sent upstream, and * 'destination' when the event is sent downstream) */ + @Deprecated public static MessageEvent messageEvent(Channel channel, ChannelFuture future, Object message, SocketAddress remoteAddress) { return new DownstreamMessageEvent(channel, future, message, remoteAddress); } diff --git a/src/main/java/org/jboss/netty/channel/DownstreamMessageEvent.java b/src/main/java/org/jboss/netty/channel/DownstreamMessageEvent.java index 2977952928..2bd4580d09 100644 --- a/src/main/java/org/jboss/netty/channel/DownstreamMessageEvent.java +++ b/src/main/java/org/jboss/netty/channel/DownstreamMessageEvent.java @@ -27,11 +27,7 @@ import java.net.SocketAddress; import org.jboss.netty.util.StringUtil; /** - * The default {@link MessageEvent} implementation. It is recommended to - * use {@link Channels#messageEvent(Channel, ChannelFuture, Object)} and - * {@link Channels#messageEvent(Channel, ChannelFuture, Object, SocketAddress)} - * to create a new {@link MessageEvent} instance rather than calling the - * constructor explicitly. + * The default {@link MessageEvent} implementation. * * @author The Netty Project (netty-dev@lists.jboss.org) * @author Trustin Lee (tlee@redhat.com) diff --git a/src/main/java/org/jboss/netty/channel/UpstreamMessageEvent.java b/src/main/java/org/jboss/netty/channel/UpstreamMessageEvent.java index 7776e0d3f2..52f53b2b15 100644 --- a/src/main/java/org/jboss/netty/channel/UpstreamMessageEvent.java +++ b/src/main/java/org/jboss/netty/channel/UpstreamMessageEvent.java @@ -29,11 +29,7 @@ import java.net.SocketAddress; import org.jboss.netty.util.StringUtil; /** - * The default {@link MessageEvent} implementation. It is recommended to - * use {@link Channels#messageEvent(Channel, ChannelFuture, Object)} and - * {@link Channels#messageEvent(Channel, ChannelFuture, Object, SocketAddress)} - * to create a new {@link MessageEvent} instance rather than calling the - * constructor explicitly. + * The default {@link MessageEvent} implementation. * * @author The Netty Project (netty-dev@lists.jboss.org) * @author Trustin Lee (tlee@redhat.com)