You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2013/01/18 15:12:57 UTC
svn commit: r1435149 -
/httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java
Author: olegk
Date: Fri Jan 18 14:12:56 2013
New Revision: 1435149
URL: http://svn.apache.org/viewvc?rev=1435149&view=rev
Log:
HTTPCORE-331: BasicFuture no longer executes notification callbacks inside a synchronized block (cleanup)
Modified:
httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java
Modified: httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java?rev=1435149&r1=1435148&r2=1435149&view=diff
==============================================================================
--- httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java (original)
+++ httpcomponents/httpcore/trunk/httpcore/src/main/java/org/apache/http/concurrent/BasicFuture.java Fri Jan 18 14:12:56 2013
@@ -99,7 +99,7 @@ public class BasicFuture<T> implements F
}
}
- public synchronized boolean completed(final T result) {
+ public boolean completed(final T result) {
synchronized(this) {
if (this.completed) {
return false;
@@ -114,7 +114,7 @@ public class BasicFuture<T> implements F
return true;
}
- public synchronized boolean failed(final Exception exception) {
+ public boolean failed(final Exception exception) {
synchronized(this) {
if (this.completed) {
return false;
@@ -129,7 +129,7 @@ public class BasicFuture<T> implements F
return true;
}
- public synchronized boolean cancel(final boolean mayInterruptIfRunning) {
+ public boolean cancel(final boolean mayInterruptIfRunning) {
synchronized(this) {
if (this.completed) {
return false;