From 9a654d8a617485cb417071f7b77d74ae181f709f Mon Sep 17 00:00:00 2001 From: Trustin Lee Date: Tue, 29 Jul 2014 15:57:58 -0700 Subject: [PATCH] Remove duplicate range check in AbstractByteBuf.skipBytes() --- .../src/main/java/io/netty/buffer/AbstractByteBuf.java | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/buffer/src/main/java/io/netty/buffer/AbstractByteBuf.java b/buffer/src/main/java/io/netty/buffer/AbstractByteBuf.java index 8b69427bb3..00654458f9 100644 --- a/buffer/src/main/java/io/netty/buffer/AbstractByteBuf.java +++ b/buffer/src/main/java/io/netty/buffer/AbstractByteBuf.java @@ -711,14 +711,7 @@ public abstract class AbstractByteBuf extends ByteBuf { @Override public ByteBuf skipBytes(int length) { checkReadableBytes(length); - - int newReaderIndex = readerIndex + length; - if (newReaderIndex > writerIndex) { - throw new IndexOutOfBoundsException(String.format( - "length: %d (expected: readerIndex(%d) + length <= writerIndex(%d))", - length, readerIndex, writerIndex)); - } - readerIndex = newReaderIndex; + readerIndex += length; return this; }