andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Mark ChannelHandler.exceptionCaught(...) as deprected.

Browse Source 
Motivation:

exceptionCaught(...) will only handle inbound exceptions which means it makes not much sense to have it also on ChannelOutboundHandler. Because of this we should move it to ChannelInboundHandler.

Modifications:

Add @deprecated annotation to ChannelHandler.exceptionCaught(...).

Result:

Preapre to cleanup the API in later release.
This commit is contained in:
Norman Maurer 2015-11-11 10:32:03 -08:00
parent 80f6104072
commit 7c5a1178b5
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
transport/src/main/java/io/netty/channel/ChannelHandler.java
View File

@ -189,7 +189,10 @@ public interface ChannelHandler {
/** /**
* Gets called if a {@link Throwable} was thrown. * Gets called if a {@link Throwable} was thrown.
*
* @deprecated is part of {@link ChannelInboundHandler}
*/ */
@Deprecated
void exceptionCaught(ChannelHandlerContext ctx, Throwable cause) throws Exception; void exceptionCaught(ChannelHandlerContext ctx, Throwable cause) throws Exception;
/** /**

1
transport/src/main/java/io/netty/channel/ChannelInboundHandler.java
View File

@ -70,5 +70,6 @@ public interface ChannelInboundHandler extends ChannelHandler {
* Gets called if a {@link Throwable} was thrown. * Gets called if a {@link Throwable} was thrown.
*/ */
@Override @Override
@SuppressWarnings("deprecated")
void exceptionCaught(ChannelHandlerContext ctx, Throwable cause) throws Exception; void exceptionCaught(ChannelHandlerContext ctx, Throwable cause) throws Exception;
} }