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

Fix missleading comment in AbstractChannelHandlerContext.invokeHandler()

Browse Source 
Motivation:

The comment on AbstractChannelHandlerContext.invokeHandler() is incorrect and missleading. See [#6177]

Modifications:

Change true to false to correct the comment.

Result:

Fix missleading and incorrect comment.
This commit is contained in:
Norman Maurer 2017-01-07 21:08:55 +01:00
parent 2457f386d8
commit c1830c8b4e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
transport/src/main/java/io/netty/channel/AbstractChannelHandlerContext.java
View File

@ -978,7 +978,7 @@ abstract class AbstractChannelHandlerContext extends DefaultAttributeMap
* Makes best possible effort to detect if {@link ChannelHandler#handlerAdded(ChannelHandlerContext)} was called * Makes best possible effort to detect if {@link ChannelHandler#handlerAdded(ChannelHandlerContext)} was called
* yet. If not return {@code false} and if called or could not detect return {@code true}. * yet. If not return {@code false} and if called or could not detect return {@code true}.
* *
* If this method returns {@code true} we will not invoke the {@link ChannelHandler} but just forward the event. * If this method returns {@code false} we will not invoke the {@link ChannelHandler} but just forward the event.
* This is needed as {@link DefaultChannelPipeline} may already put the {@link ChannelHandler} in the linked-list * This is needed as {@link DefaultChannelPipeline} may already put the {@link ChannelHandler} in the linked-list
* but not called {@link ChannelHandler#handlerAdded(ChannelHandlerContext)}. * but not called {@link ChannelHandler#handlerAdded(ChannelHandlerContext)}.
*/ */