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

Fix SctpMessage.duplicate() behavior

Browse Source 
Motivation:

SctpMessage.duplicate() copied message content that leads to additional buffer
allocation and memory copying.

Modifications:

Duplicate message content instead of copying it.

Result:

Better performace and less memory consumption.
This commit is contained in:
Sergey Polovko 2016-01-05 00:40:35 +03:00 committed by Norman Maurer
parent bdea94d807
commit 968ef8e0af
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
transport-sctp/src/main/java/io/netty/channel/sctp/SctpMessage.java
View File

@ -164,7 +164,7 @@ public final class SctpMessage extends DefaultByteBufHolder {
if (msgInfo == null) { if (msgInfo == null) {
return new SctpMessage(protocolIdentifier, streamIdentifier, unordered, content().duplicate()); return new SctpMessage(protocolIdentifier, streamIdentifier, unordered, content().duplicate());
} else { } else {
return new SctpMessage(msgInfo, content().copy()); return new SctpMessage(msgInfo, content().duplicate());
} }
} }