------------------------------------------------------------
revno: 11793
revision-id: squid3@treenet.co.nz-20130301110557-p7073w1flufpo4b2
parent: squid3@treenet.co.nz-20130301110508-xh47ed92p2w3pumx
committer: Amos Jeffries <squid3@treenet.co.nz>
branch nick: 3.2
timestamp: Fri 2013-03-01 04:05:57 -0700
message:
  Docs: document ConnOpener::swanSong() better
------------------------------------------------------------
# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: squid3@treenet.co.nz-20130301110557-p7073w1flufpo4b2
# target_branch: http://bzr.squid-cache.org/bzr/squid3/branches\
#   /SQUID_3_2
# testament_sha1: a525861cebe18e91e8522d03d7c8083f50c20d5f
# timestamp: 2013-03-01 11:06:44 +0000
# source_branch: http://bzr.squid-cache.org/bzr/squid3/branches\
#   /SQUID_3_2
# base_revision_id: squid3@treenet.co.nz-20130301110508-\
#   xh47ed92p2w3pumx
# 
# Begin patch
=== modified file 'src/comm/ConnOpener.cc'
--- src/comm/ConnOpener.cc	2013-02-09 06:36:36 +0000
+++ src/comm/ConnOpener.cc	2013-03-01 11:05:57 +0000
@@ -57,9 +57,11 @@
         sendAnswer(COMM_ERR_CONNECT, 0, "Comm::ConnOpener::swanSong");
     }
 
+    // did we abort with a temporary FD assigned?
     if (temporaryFd_ >= 0)
         closeFd();
 
+    // did we abort while waiting between retries?
     if (calls_.sleep_)
         cancelSleep();
 
@@ -110,6 +112,7 @@
         if (callback_->canceled()) {
             debugs(5, 4, conn_ << " not calling canceled " << *callback_ <<
                    " [" << callback_->id << ']' );
+            // TODO save the pconn to the pconnPool ?
         } else {
             typedef CommConnectCbParams Params;
             Params &params = GetCommParams<Params>(callback_);