From 5d6ef046cdfe06ed4e3345a4f0f9988ed0a1fa91 Mon Sep 17 00:00:00 2001 From: Trustin Lee Date: Tue, 6 Jan 2009 05:32:44 +0000 Subject: [PATCH] Removed unnecessary 'else' --- .../handler/codec/frame/LengthFieldBasedFrameDecoder.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/main/java/org/jboss/netty/handler/codec/frame/LengthFieldBasedFrameDecoder.java b/src/main/java/org/jboss/netty/handler/codec/frame/LengthFieldBasedFrameDecoder.java index 34ae4e48c4..0d1abee373 100644 --- a/src/main/java/org/jboss/netty/handler/codec/frame/LengthFieldBasedFrameDecoder.java +++ b/src/main/java/org/jboss/netty/handler/codec/frame/LengthFieldBasedFrameDecoder.java @@ -160,7 +160,9 @@ public class LengthFieldBasedFrameDecoder extends FrameDecoder { throw new CorruptedFrameException( "Adjusted length (" + frameLength + ") is less than " + lengthFieldEndOffset); - } else if (frameLength > maxFrameLength) { + } + + if (frameLength > maxFrameLength) { // Enter the discard mode and discard everything received so far. discardingTooLongFrame = true; tooLongFrameLength = frameLength; @@ -173,8 +175,8 @@ public class LengthFieldBasedFrameDecoder extends FrameDecoder { int frameLengthInt = (int) frameLength; if (buffer.readableBytes() < frameLengthInt) { return null; - } else { - return buffer.readBytes(frameLengthInt); } + + return buffer.readBytes(frameLengthInt); } }