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

Reply via email to