https://bz.apache.org/bugzilla/show_bug.cgi?id=58342
Bug ID: 58342 Summary: org.apache.tomcat.websocket.WsSession has repeated conditional tests Product: Tomcat 9 Version: unspecified Hardware: PC Status: NEW Severity: minor Priority: P2 Component: WebSocket Assignee: dev@tomcat.apache.org Reporter: dinesh.a.jo...@gmail.com Created attachment 33078 --> https://bz.apache.org/bugzilla/attachment.cgi?id=33078&action=edit Patch file In the method WsSession.removeMessageHandler(), two `if` statements have repeated conditional tests. I did not investigate whether older versions of Tomcat have it however they're present in trunk. I am attaching a patch to fix it. I ran the tests and they passed with my changes. -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org