In org.apache.tomcat.websocket.pojo.PojoMethodMapping, there are cases like this: if (currentClazz == clazzPojo || (currentClazz != clazzPojo && !isMethodOverride(open, method))) { The clause 'currentClazz != clazzPojo' is redundant since we know 'currentClazz == clazzPojo' is false. This is repeated several times.
Created attachment 33208 [details] Patch to remove repeated conditional tests
Created attachment 33209 [details] Patch for similar problem for AprEndpoint I discovered the same problem with org.apache.tomcat.util.net.AprEndpoint, so please see the second patch too.
Hi, Thanks for the report and the patch. The fix is available in trunk, 8.0.x (for 8.0.29 onwards) and 7.0.x (for 7.0.66 onwards) Regards, Violeta