You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2013/02/16 17:13:24 UTC
svn commit: r1446910 -
/tomcat/trunk/java/org/apache/tomcat/websocket/WsRemoteEndpointBase.java
Author: markt
Date: Sat Feb 16 16:13:24 2013
New Revision: 1446910
URL: http://svn.apache.org/r1446910
Log:
Fix Findbugs warning
Modified:
tomcat/trunk/java/org/apache/tomcat/websocket/WsRemoteEndpointBase.java
Modified: tomcat/trunk/java/org/apache/tomcat/websocket/WsRemoteEndpointBase.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/WsRemoteEndpointBase.java?rev=1446910&r1=1446909&r2=1446910&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/websocket/WsRemoteEndpointBase.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/WsRemoteEndpointBase.java Sat Feb 16 16:13:24 2013
@@ -667,7 +667,7 @@ public abstract class WsRemoteEndpointBa
@Override
public SendResult get() throws InterruptedException,
- ExecutionException {
+ ExecutionException {
latch.await();
return result;
}
@@ -676,7 +676,10 @@ public abstract class WsRemoteEndpointBa
public SendResult get(long timeout, TimeUnit unit)
throws InterruptedException, ExecutionException,
TimeoutException {
- latch.await(timeout, unit);
+ boolean retval = latch.await(timeout, unit);
+ if (retval == false) {
+ throw new TimeoutException();
+ }
return result;
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org