ASF Bugzilla – Attachment 33078 Details for
Bug 58342
org.apache.tomcat.websocket.WsSession has repeated conditional tests
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
[patch]
Patch file
0001-Fix-repeated-conditionals.patch (text/plain), 1.12 KB, created by
DJ
on 2015-09-07 17:58:56 UTC
(
hide
)
Description:
Patch file
Filename:
MIME Type:
Creator:
DJ
Created:
2015-09-07 17:58:56 UTC
Size:
1.12 KB
patch
obsolete
>From aed9c7f64baab2edf1f6f94d4435b8fad415adbc Mon Sep 17 00:00:00 2001 >From: dineshj <dinesh.joshi@yahoo.com> >Date: Sun, 6 Sep 2015 12:20:47 -0700 >Subject: [PATCH] Fix repeated conditionals > >--- > java/org/apache/tomcat/websocket/WsSession.java | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/java/org/apache/tomcat/websocket/WsSession.java b/java/org/apache/tomcat/websocket/WsSession.java >index 7994cef..aec20f3 100644 >--- a/java/org/apache/tomcat/websocket/WsSession.java >+++ b/java/org/apache/tomcat/websocket/WsSession.java >@@ -315,13 +315,13 @@ public class WsSession implements Session { > removed = true; > } > >- if (listener.equals(binaryMessageHandler) || >+ if (wrapped.equals(binaryMessageHandler) || > listener.equals(binaryMessageHandler)) { > binaryMessageHandler = null; > removed = true; > } > >- if (listener.equals(pongMessageHandler) || >+ if (wrapped.equals(pongMessageHandler) || > listener.equals(pongMessageHandler)) { > pongMessageHandler = null; > removed = true; >-- >2.2.1 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 58342
: 33078