You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hc.apache.org by se...@apache.org on 2009/03/19 21:41:47 UTC

svn commit: r756183 - /httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java

Author: sebb
Date: Thu Mar 19 20:41:46 2009
New Revision: 756183

URL: http://svn.apache.org/viewvc?rev=756183&view=rev
Log:
Make schemeRegistry final. This makes the class ThreadSafe

Modified:
    httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java

Modified: httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java?rev=756183&r1=756182&r2=756183&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java (original)
+++ httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java Thu Mar 19 20:41:46 2009
@@ -36,6 +36,8 @@
 import java.net.Socket;
 import java.net.InetAddress;
 
+import net.jcip.annotations.ThreadSafe;
+
 import org.apache.http.HttpHost;
 import org.apache.http.params.HttpParams;
 import org.apache.http.params.HttpConnectionParams;
@@ -63,12 +65,13 @@
  *
  * @since 4.0
  */
+@ThreadSafe
 public class DefaultClientConnectionOperator
     implements ClientConnectionOperator {
 
 
     /** The scheme registry for looking up socket factories. */
-    protected SchemeRegistry schemeRegistry;
+    protected final SchemeRegistry schemeRegistry; // @ThreadSafe
 
 
     /**