svn commit: r1426274 - /tomcat/trunk/java/org/apache/tomcat/websocket/PojoEndpointConfiguration.java

[ma...@apache.org]

Author: markt
Date: Thu Dec 27 17:48:56 2012
New Revision: 1426274

URL: http://svn.apache.org/viewvc?rev=1426274&view=rev
Log:
Formatting, javadoc and the remaining TODO

Modified:
    tomcat/trunk/java/org/apache/tomcat/websocket/PojoEndpointConfiguration.java

Modified: tomcat/trunk/java/org/apache/tomcat/websocket/PojoEndpointConfiguration.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/PojoEndpointConfiguration.java?rev=1426274&r1=1426273&r2=1426274&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/websocket/PojoEndpointConfiguration.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/PojoEndpointConfiguration.java Thu Dec 27 17:48:56 2012
@@ -18,20 +18,19 @@ package org.apache.tomcat.websocket;
 
 import javax.websocket.server.DefaultServerConfiguration;
 
-public class PojoEndpointConfiguration extends
-        DefaultServerConfiguration {
+/**
+ * Provides the configuration for POJOs annotated at WebSocket endpoints. It
+ * provides the means, via casting, of new {@link WsEndpointPojo} instances
+ * obtaining POJO endpoint specific configuration settings such as the mapping
+ * of onXxx calls to POJO methods.
+ */
+public class PojoEndpointConfiguration extends DefaultServerConfiguration {
 
     private final Class<?> pojoClass;
     private final PojoMethodMapping methodMapping;
     private final String servletPath;
     private final String pathInfo;
 
-    @Override
-    public boolean checkOrigin(String originHeaderValue) {
-        // Allow all
-        return true;
-    }
-
 
     PojoEndpointConfiguration(Class<?> pojoClass,
             PojoMethodMapping methodMapping, String servletPath,
@@ -45,16 +44,22 @@ public class PojoEndpointConfiguration e
 
 
     @Override
+    public boolean checkOrigin(String originHeaderValue) {
+        // Allow all
+        return true;
+    }
+
+
+    @Override
     public String getPath() {
         return servletPath;
     }
 
+
     public Object getPojo() {
         try {
             return pojoClass.newInstance();
         } catch (InstantiationException | IllegalAccessException e) {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
             throw new IllegalArgumentException(e);
         }
     }



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