Author: markt
Date: Mon Mar 25 17:50:00 2013
New Revision: 1460781
URL: http://svn.apache.org/r1460781
Log:
Code review while looking at
https://issues.apache.org/bugzilla/show_bug.cgi?id=54746
No need for this. The container handles it (and now protects against recursive
calls if an endpoint does do this).
Modified:
tomcat/trunk/webapps/examples/WEB-INF/classes/websocket/echo/EchoEndpoint.java
Modified:
tomcat/trunk/webapps/examples/WEB-INF/classes/websocket/echo/EchoEndpoint.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/webapps/examples/WEB-INF/classes/websocket/echo/EchoEndpoint.java?rev=1460781&r1=1460780&r2=1460781&view=diff
==============================================================================
---
tomcat/trunk/webapps/examples/WEB-INF/classes/websocket/echo/EchoEndpoint.java
(original)
+++
tomcat/trunk/webapps/examples/WEB-INF/classes/websocket/echo/EchoEndpoint.java
Mon Mar 25 17:50:00 2013
@@ -18,8 +18,6 @@ package websocket.echo;
import java.io.IOException;
-import javax.websocket.CloseReason;
-import javax.websocket.CloseReason.CloseCodes;
import javax.websocket.Endpoint;
import javax.websocket.EndpointConfig;
import javax.websocket.MessageHandler;
@@ -34,16 +32,6 @@ public class EchoEndpoint extends Endpoi
session.addMessageHandler(new EchoMessageHandler(remoteEndpointBasic));
}
- @Override
- public void onClose(Session session, CloseReason closeReason) {
- try {
- session.close(new CloseReason(CloseCodes.NORMAL_CLOSURE, null));
- } catch (IOException e) {
- // TODO Auto-generated catch block
- e.printStackTrace();
- }
- }
-
private static class EchoMessageHandler
implements MessageHandler.Whole<String> {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
