2013/3/25  <ma...@apache.org>:
> Author: markt
> Date: Mon Mar 25 15:01:41 2013
> New Revision: 1460700
>
> URL: http://svn.apache.org/r1460700
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54740
> Update SCI scan to reflect changes in spec
>
> Modified:
>     tomcat/trunk/java/org/apache/tomcat/websocket/server/WsSci.java
>
> Modified: tomcat/trunk/java/org/apache/tomcat/websocket/server/WsSci.java
> URL: 
> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/server/WsSci.java?rev=1460700&r1=1460699&r2=1460700&view=diff

(...)
> @@ -60,6 +60,12 @@ public class WsSci implements ServletCon
>              String wsPackage = ContainerProvider.class.getName();
>              wsPackage = wsPackage.substring(0, wsPackage.lastIndexOf('.') + 
> 1);
>              for (Class<?> clazz : clazzes) {
> +                int modifiers = clazz.getModifiers();
> +                if (!Modifier.isPublic(clazz.getModifiers()) ||

The value has already been assigned to a variable.
s/ isPublic(clazz.getModifiers()) / isPublic(modifiers) /

> +                        Modifier.isAbstract(modifiers)) {
> +                    // Non-public or abstract - skip it.
> +                    continue;
> +                }
>                  // Protect against scanning the WebSocket API JARs
>                  if (clazz.getName().startsWith(wsPackage)) {
>                      continue;
> @@ -68,14 +74,11 @@ public class WsSci implements ServletCon


Best regards,
Konstantin Kolinko

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

Reply via email to