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

cleanup, duplicated static final fields

Browse Source 
Motivation:

There are few duplicated byte[] CRLF fields in code.

Modifications:

Removed duplicated fields as they could be inherited from parent encoder.

Result:

Less static fields.
This commit is contained in:
Dmitriy Dumanskiy 2016-08-09 10:44:31 +03:00 committed by Norman Maurer
parent e0331e6ccb
commit aef8de5ec1
3 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
codec-http/src/main/java/io/netty/handler/codec/http/HttpObjectEncoder.java
View File

@ -41,7 +41,7 @@ import static io.netty.handler.codec.http.HttpConstants.*;
* implement all abstract methods properly.
*/
public abstract class HttpObjectEncoder<H extends HttpMessage> extends MessageToMessageEncoder<Object> {
private static final byte[] CRLF = { CR, LF };
static final byte[] CRLF = { CR, LF };
private static final byte[] ZERO_CRLF = { '0', CR, LF };
private static final byte[] ZERO_CRLF_CRLF = { '0', CR, LF, CR, LF };
private static final ByteBuf CRLF_BUF = unreleasableBuffer(directBuffer(CRLF.length).writeBytes(CRLF));

1
codec-http/src/main/java/io/netty/handler/codec/http/HttpRequestEncoder.java
View File

@ -27,7 +27,6 @@ import static io.netty.handler.codec.http.HttpConstants.*;
public class HttpRequestEncoder extends HttpObjectEncoder<HttpRequest> {
private static final char SLASH = '/';
private static final char QUESTION_MARK = '?';
private static final byte[] CRLF = { CR, LF };
@Override
public boolean acceptOutboundMessage(Object msg) throws Exception {

1
codec-http/src/main/java/io/netty/handler/codec/http/HttpResponseEncoder.java
View File

@ -24,7 +24,6 @@ import static io.netty.handler.codec.http.HttpConstants.*;
* a {@link ByteBuf}.
*/
public class HttpResponseEncoder extends HttpObjectEncoder<HttpResponse> {
private static final byte[] CRLF = { CR, LF };
@Override
public boolean acceptOutboundMessage(Object msg) throws Exception {