On 25/03/2013 15:07, Konstantin Kolinko wrote: > 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) /
Thanks. Too much copy and paste. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
