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

Replace a variable with a constant wherever possible

Browse Source
This commit is contained in:
Trustin Lee 2012-11-12 09:43:14 +09:00
parent 12115198d0
commit d23766fa27
3 changed files with 13 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/org/jboss/netty/handler/codec/http/multipart/HttpPostRequestDecoder.java
View File

@ -1339,11 +1339,11 @@ public class HttpPostRequestDecoder {
} else if (nextByte == HttpConstants.LF) { } else if (nextByte == HttpConstants.LF) {
return sb.toString(); return sb.toString();
} else if (nextByte == '-') { } else if (nextByte == '-') {
sb.append((char) nextByte); sb.append('-');
// second check for closing delimiter // second check for closing delimiter
nextByte = undecodedChunk.readByte(); nextByte = undecodedChunk.readByte();
if (nextByte == '-') { if (nextByte == '-') {
sb.append((char) nextByte); sb.append('-');
// now try to find if CRLF or LF there // now try to find if CRLF or LF there
if (undecodedChunk.readable()) { if (undecodedChunk.readable()) {
nextByte = undecodedChunk.readByte(); nextByte = undecodedChunk.readByte();
@ -1439,12 +1439,12 @@ public class HttpPostRequestDecoder {
sao.setReadPosition(0); sao.setReadPosition(0);
return sb.toString(); return sb.toString();
} else if (nextByte == '-') { } else if (nextByte == '-') {
sb.append((char) nextByte); sb.append('-');
// second check for closing delimiter // second check for closing delimiter
if (sao.pos < sao.limit) { if (sao.pos < sao.limit) {
nextByte = sao.bytes[sao.pos ++]; nextByte = sao.bytes[sao.pos ++];
if (nextByte == '-') { if (nextByte == '-') {
sb.append((char) nextByte); sb.append('-');
// now try to find if CRLF or LF there // now try to find if CRLF or LF there
if (sao.pos < sao.limit) { if (sao.pos < sao.limit) {
nextByte = sao.bytes[sao.pos++]; nextByte = sao.bytes[sao.pos++];

14
src/main/java/org/jboss/netty/handler/codec/http/websocketx/WebSocketClientHandshakerFactory.java
View File

@ -18,6 +18,8 @@ package org.jboss.netty.handler.codec.http.websocketx;
import java.net.URI; import java.net.URI;
import java.util.Map; import java.util.Map;
import static org.jboss.netty.handler.codec.http.websocketx.WebSocketVersion.*;
/** /**
* Instances the appropriate handshake class to use for clients * Instances the appropriate handshake class to use for clients
*/ */
@ -66,17 +68,17 @@ public class WebSocketClientHandshakerFactory {
public WebSocketClientHandshaker newHandshaker( public WebSocketClientHandshaker newHandshaker(
URI webSocketURL, WebSocketVersion version, String subprotocol, URI webSocketURL, WebSocketVersion version, String subprotocol,
boolean allowExtensions, Map<String, String> customHeaders, long maxFramePayloadLength) { boolean allowExtensions, Map<String, String> customHeaders, long maxFramePayloadLength) {
if (version == WebSocketVersion.V13) { if (version == V13) {
return new WebSocketClientHandshaker13( return new WebSocketClientHandshaker13(
webSocketURL, version, subprotocol, allowExtensions, customHeaders, maxFramePayloadLength); webSocketURL, V13, subprotocol, allowExtensions, customHeaders, maxFramePayloadLength);
} }
if (version == WebSocketVersion.V08) { if (version == V08) {
return new WebSocketClientHandshaker08( return new WebSocketClientHandshaker08(
webSocketURL, version, subprotocol, allowExtensions, customHeaders, maxFramePayloadLength); webSocketURL, V08, subprotocol, allowExtensions, customHeaders, maxFramePayloadLength);
} }
if (version == WebSocketVersion.V00) { if (version == V00) {
return new WebSocketClientHandshaker00( return new WebSocketClientHandshaker00(
webSocketURL, version, subprotocol, customHeaders, maxFramePayloadLength); webSocketURL, V00, subprotocol, customHeaders, maxFramePayloadLength);
} }
throw new WebSocketHandshakeException("Protocol version " + version.toString() + " not supported."); throw new WebSocketHandshakeException("Protocol version " + version.toString() + " not supported.");

2
src/main/java/org/jboss/netty/util/internal/jzlib/InfBlocks.java
View File

@ -259,7 +259,7 @@ final class InfBlocks {
if (n == 0) { if (n == 0) {
bitb = b; bitb = b;
bitk = k; bitk = k;
z.avail_in = n; z.avail_in = 0;
z.total_in += p - z.next_in_index; z.total_in += p - z.next_in_index;
z.next_in_index = p; z.next_in_index = p;
write = q; write = q;