Author: markt
Date: Tue Feb 28 20:20:12 2012
New Revision: 1294817
URL: http://svn.apache.org/viewvc?rev=1294817&view=rev
Log:
Remove unnecessary comment post refactoring.
Fix new lines.
Modified:
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeAprProcessor.java
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeBioProcessor.java
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeNioProcessor.java
Modified:
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeAprProcessor.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeAprProcessor.java?rev=1294817&r1=1294816&r2=1294817&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeAprProcessor.java
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeAprProcessor.java
Tue Feb 28 20:20:12 2012
@@ -21,12 +21,6 @@ import java.io.IOException;
import org.apache.tomcat.jni.Socket;
import org.apache.tomcat.util.net.SocketWrapper;
-/**
- * Implementation note: The need to extend Http11Processor could probably be
- * removed if the Processor interface was expanded to cover all of the methods
- * required by the AbstractProtocol. That would simplify the code and further
- * reduce the size of instances of this class.
- */
public class UpgradeAprProcessor extends UpgradeProcessor<Long> {
private final long socket;
Modified:
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeBioProcessor.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeBioProcessor.java?rev=1294817&r1=1294816&r2=1294817&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeBioProcessor.java
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeBioProcessor.java
Tue Feb 28 20:20:12 2012
@@ -23,12 +23,6 @@ import java.net.Socket;
import org.apache.tomcat.util.net.SocketWrapper;
-/**
- * Implementation note: The need to extend Http11Processor could probably be
- * removed if the Processor interface was expanded to cover all of the methods
- * required by the AbstractProtocol. That would simplify the code and further
- * reduce the size of instances of this class.
- */
public class UpgradeBioProcessor extends UpgradeProcessor<Socket> {
private final InputStream inputStream;
Modified:
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeNioProcessor.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeNioProcessor.java?rev=1294817&r1=1294816&r2=1294817&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeNioProcessor.java
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeNioProcessor.java
Tue Feb 28 20:20:12 2012
@@ -25,12 +25,6 @@ import org.apache.tomcat.util.net.NioEnd
import org.apache.tomcat.util.net.NioSelectorPool;
import org.apache.tomcat.util.net.SocketWrapper;
-/**
- * Implementation note: The need to extend Http11Processor could probably be
- * removed if the Processor interface was expanded to cover all of the methods
- * required by the AbstractProtocol. That would simplify the code and further
- * reduce the size of instances of this class.
- */
public class UpgradeNioProcessor extends UpgradeProcessor<NioChannel> {
private final NioChannel nioChannel;
@@ -104,7 +98,8 @@ public class UpgradeNioProcessor extends
return -1;
} else {
return bytes[0] & 0xFF;
- } }
+ }
+ }
@Override
public int read(byte[] bytes, int off, int len) throws IOException {
@@ -113,7 +108,6 @@ public class UpgradeNioProcessor extends
} else {
return readSocket(true, bytes, off, len);
}
-
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
