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

Remove unnecessary try-catch block in AbstractUnsafe.flush()

Browse Source
This commit is contained in:
Trustin Lee 2013-07-12 20:00:30 +09:00
parent 43f12569aa
commit e2fcb06d9a
1 changed files with 5 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
transport/src/main/java/io/netty/channel/AbstractChannel.java
View File

@ -597,16 +597,11 @@ public abstract class AbstractChannel extends DefaultAttributeMap implements Cha
outboundBuffer.addFlush();
if (!inFlushNow) { // Avoid re-entrance
try {
// Flush immediately only when there's no pending flush.
// If there's a pending flush operation, event loop will call flushNow() later,
// and thus there's no need to call it now.
if (!isFlushPending()) {
flushNow();
}
} catch (Throwable t) {
outboundBuffer.fail(t);
close(voidPromise());
// Flush immediately only when there's no pending flush.
// If there's a pending flush operation, event loop will call flushNow() later,
// and thus there's no need to call it now.
if (!isFlushPending()) {
flushNow();
}
} else {
if (!flushNowPending) {