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

Correctly shutdown wrapped Executor in PausableChannelEventExecutor.shutdown()

Browse Source 
Motivation:

PausableChannelEventExecutor().shutdown() used to call unwrap().terminationFuture() instead of unwrap.shutdown().
This error was reported by @xfrag in a comment to a commit message [1]. Tyvm.

Modifications:

PausableChannelEventExecutor.shutdown() now correctly invokes unwrap().shutdown() instead of unwrap().terminationFuture().

Result:

Correct code.

[1] 220660e351 (commitcomment-8489643)
This commit is contained in:
Jakob Buchgraber 2014-11-10 19:35:13 +01:00 committed by Norman Maurer
parent 009b612934
commit 229f83a7e6
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/PausableChannelEventExecutor.java
View File

@ -200,7 +200,7 @@ abstract class PausableChannelEventExecutor implements PausableEventExecutor, Ch
@Override
@Deprecated
public void shutdown() {
unwrap().terminationFuture();
unwrap().shutdown();
}
@Override