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

Remove the use of ThreadLocal for performance reasons

Browse Source
This commit is contained in:
Norman Maurer 2013-11-29 15:57:31 +01:00
parent 3f7b674db8
commit 818bc602ee
1 changed files with 5 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
codec-http/src/main/java/io/netty/handler/codec/http/HttpObjectDecoder.java
View File

@ -100,13 +100,6 @@ import static io.netty.buffer.ByteBufUtil.readBytes;
*/
public abstract class HttpObjectDecoder extends ReplayingDecoder<HttpObjectDecoder.State> {
private static final ThreadLocal<StringBuilder> BUILDERS = new ThreadLocal<StringBuilder>() {
@Override
protected StringBuilder initialValue() {
return new StringBuilder(512);
}
};
private final int maxInitialLineLength;
private final int maxHeaderSize;
private final int maxChunkSize;
@ -118,7 +111,7 @@ public abstract class HttpObjectDecoder extends ReplayingDecoder<HttpObjectDecod
private long chunkSize;
private int headerSize;
private int contentRead;
private StringBuilder sb;
private final StringBuilder sb = new StringBuilder(128);
/**
* The internal state of {@link HttpObjectDecoder}.
@ -667,7 +660,8 @@ public abstract class HttpObjectDecoder extends ReplayingDecoder<HttpObjectDecod
}
private StringBuilder readHeader(ByteBuf buffer) {
StringBuilder sb = builder();
StringBuilder sb = this.sb;
sb.setLength(0);
int headerSize = this.headerSize;
loop:
@ -723,7 +717,8 @@ public abstract class HttpObjectDecoder extends ReplayingDecoder<HttpObjectDecod
}
private StringBuilder readLine(ByteBuf buffer, int maxLineLength) {
StringBuilder sb = builder();
StringBuilder sb = this.sb;
sb.setLength(0);
int lineLength = 0;
while (true) {
byte nextByte = buffer.readByte();
@ -840,14 +835,4 @@ public abstract class HttpObjectDecoder extends ReplayingDecoder<HttpObjectDecod
}
return result;
}
private StringBuilder builder() {
if (sb == null) {
// Obtain the StringBuilder from the ThreadLocal and store it for later usage.
// This minimize the ThreadLocal.get() operations a lot and so eliminate some overhead
sb = BUILDERS.get();
}
sb.setLength(0);
return sb;
}
}