Remove extra checks to fix #9456 (#9523)

Motivation:

There are some extra log level checks (logger.isWarnEnabled()).

Modification:

Remove log level checks (logger.isWarnEnabled()) from io.netty.channel.epoll.AbstractEpollStreamChannel, io.netty.channel.DefaultFileRegion, io.netty.channel.nio.AbstractNioChannel, io.netty.util.HashedWheelTimer, io.netty.handler.stream.ChunkedWriteHandler and io.netty.channel.udt.nio.NioUdtMessageConnectorChannel

Result:

Fixes #9456
This commit is contained in:
Xiaoqin Fu 2019-08-30 01:37:30 -07:00 committed by Norman Maurer
parent 291f80733a
commit 21b7e29ea7
6 changed files with 6 additions and 18 deletions

View File

@ -270,10 +270,8 @@ public class HashedWheelTimer implements Timer {
}
if (duration < MILLISECOND_NANOS) {
if (logger.isWarnEnabled()) {
logger.warn("Configured tickDuration {} smaller then {}, using 1ms.",
tickDuration, MILLISECOND_NANOS);
}
this.tickDuration = MILLISECOND_NANOS;
} else {
this.tickDuration = duration;

View File

@ -119,11 +119,9 @@ public class ChunkedWriteHandler extends ChannelDuplexHandler {
try {
doFlush(ctx);
} catch (Exception e) {
if (logger.isWarnEnabled()) {
logger.warn("Unexpected exception while sending chunks.", e);
}
}
}
@Override
public void write(ChannelHandlerContext ctx, Object msg, ChannelPromise promise) throws Exception {

View File

@ -701,12 +701,10 @@ public abstract class AbstractEpollStreamChannel extends AbstractEpollChannel im
try {
fd.close();
} catch (IOException e) {
if (logger.isWarnEnabled()) {
logger.warn("Error while closing a pipe", e);
}
}
}
}
class EpollStreamUnsafe extends AbstractEpollUnsafe {
// Overridden here just to be able to access this method from AbstractEpollStreamChannel

View File

@ -79,10 +79,8 @@ public class NioUdtMessageConnectorChannel extends AbstractNioMessageChannel imp
try {
channelUDT.close();
} catch (final Exception e2) {
if (logger.isWarnEnabled()) {
logger.warn("Failed to close channel.", e2);
}
}
throw new ChannelException("Failed to configure channel.", e);
}
}

View File

@ -161,11 +161,9 @@ public class DefaultFileRegion extends AbstractReferenceCounted implements FileR
try {
file.close();
} catch (IOException e) {
if (logger.isWarnEnabled()) {
logger.warn("Failed to close a file.", e);
}
}
}
@Override
public FileRegion retain() {

View File

@ -86,11 +86,9 @@ public abstract class AbstractNioChannel extends AbstractChannel {
try {
ch.close();
} catch (IOException e2) {
if (logger.isWarnEnabled()) {
logger.warn(
"Failed to close a partially initialized socket.", e2);
}
}
throw new ChannelException("Failed to enter non-blocking mode.", e);
}