Use ByteBuf.readSlice(...).retain() to minimize memory copies.

Motivation:
At the moment we call ByteBuf.readBytes(...) in these handlers but with optimizations done as part of 25e0d9d we can just use readSlice(...).retain() and eliminate the memory copy.

Modifications:
Replace ByteBuf.readBytes(...) usage with readSlice(...).retain().

Result:
Less memory copies.
This commit is contained in:
Norman Maurer 2014-05-09 08:31:07 +02:00
parent dd0782990b
commit e26bbfd4a7
3 changed files with 7 additions and 7 deletions

View File

@ -268,13 +268,13 @@ public class DelimiterBasedFrameDecoder extends ByteToMessageDecoder {
} }
if (stripDelimiter) { if (stripDelimiter) {
frame = buffer.readBytes(minFrameLength); frame = buffer.readSlice(minFrameLength);
buffer.skipBytes(minDelimLength); buffer.skipBytes(minDelimLength);
} else { } else {
frame = buffer.readBytes(minFrameLength + minDelimLength); frame = buffer.readSlice(minFrameLength + minDelimLength);
} }
return frame; return frame.retain();
} else { } else {
if (!discardingTooLongFrame) { if (!discardingTooLongFrame) {
if (buffer.readableBytes() > maxFrameLength) { if (buffer.readableBytes() > maxFrameLength) {

View File

@ -74,7 +74,7 @@ public class FixedLengthFrameDecoder extends ByteToMessageDecoder {
if (in.readableBytes() < frameLength) { if (in.readableBytes() < frameLength) {
return null; return null;
} else { } else {
return in.readBytes(frameLength); return in.readSlice(frameLength).retain();
} }
} }
} }

View File

@ -100,13 +100,13 @@ public class LineBasedFrameDecoder extends ByteToMessageDecoder {
} }
if (stripDelimiter) { if (stripDelimiter) {
frame = buffer.readBytes(length); frame = buffer.readSlice(length);
buffer.skipBytes(delimLength); buffer.skipBytes(delimLength);
} else { } else {
frame = buffer.readBytes(length + delimLength); frame = buffer.readSlice(length + delimLength);
} }
return frame; return frame.retain();
} else { } else {
final int length = buffer.readableBytes(); final int length = buffer.readableBytes();
if (length > maxLength) { if (length > maxLength) {