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

Posted to commits@hc.apache.org by ro...@apache.org on 2007/02/21 16:14:30 UTC

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

Author: rolandw
Date: Wed Feb 21 07:14:29 2007
New Revision: 510051

URL: http://svn.apache.org/viewvc?view=rev&rev=510051
Log:
slightly revised take on the default port support

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

Modified: jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java
URL: http://svn.apache.org/viewvc/jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java?view=diff&rev=510051&r1=510050&r2=510051
==============================================================================
--- jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java (original)
+++ jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java Wed Feb 21 07:14:29 2007
@@ -135,9 +135,8 @@
         conn.announce(sock);
 
         sock = sf.connectSocket
-            (sock, target.getHostName(), 
-                    target.getPort() == -1 ? schm.getDefaultPort() : target.getPort(), 
-                    local, 0, params);
+            (sock, target.getHostName(), target.resolvePort(),
+             local, 0, params);
         prepareSocket(sock, context, params);
 
         final boolean secure = sf.isSecure(sock);