You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2017/07/06 09:05:21 UTC

svn commit: r1800998 - /tomcat/tc8.0.x/trunk/test/org/apache/tomcat/websocket/server/TesterWsCloseClient.java

Author: markt
Date: Thu Jul  6 09:05:20 2017
New Revision: 1800998

URL: http://svn.apache.org/viewvc?rev=1800998&view=rev
Log:
Fix one FindBugs and one Javadoc warning

Modified:
    tomcat/tc8.0.x/trunk/test/org/apache/tomcat/websocket/server/TesterWsCloseClient.java

Modified: tomcat/tc8.0.x/trunk/test/org/apache/tomcat/websocket/server/TesterWsCloseClient.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/test/org/apache/tomcat/websocket/server/TesterWsCloseClient.java?rev=1800998&r1=1800997&r2=1800998&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/test/org/apache/tomcat/websocket/server/TesterWsCloseClient.java (original)
+++ tomcat/tc8.0.x/trunk/test/org/apache/tomcat/websocket/server/TesterWsCloseClient.java Thu Jul  6 09:05:20 2017
@@ -66,8 +66,9 @@ public class TesterWsCloseClient {
     private void readUpgradeResponse() throws IOException {
         BufferedReader in = new BufferedReader(new InputStreamReader(
                 socket.getInputStream()));
-        while (!in.readLine().isEmpty()) {
-
+        String line = in.readLine();
+        while (line != null && !line.isEmpty()) {
+            line = in.readLine();
         }
     }
 
@@ -78,7 +79,7 @@ public class TesterWsCloseClient {
         socket.close();
     }
 
-    /**
+    /*
      * Send a TCP RST instead of a TCP closing handshake
      */
     public void forceCloseSocket() throws IOException {



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org