[or-cvs] r18383: {tor} make a note about a clause in connection_edge.c that is prob (tor/trunk/src/or)

[arma at seul.org]

Author: arma
Date: 2009-02-04 03:27:36 -0500 (Wed, 04 Feb 2009)
New Revision: 18383

Modified:
   tor/trunk/src/or/connection_edge.c
Log:
make a note about a clause in connection_edge.c that is probably
redundant, and is definitely confusing. we should take it out
in 0.2.2.x and see who squeaks.


Modified: tor/trunk/src/or/connection_edge.c
===================================================================
--- tor/trunk/src/or/connection_edge.c	2009-02-04 07:33:53 UTC (rev 18382)
+++ tor/trunk/src/or/connection_edge.c	2009-02-04 08:27:36 UTC (rev 18383)
@@ -409,6 +409,8 @@
       ? LOG_INFO : LOG_NOTICE;
     seconds_idle = (int)( now - conn->_base.timestamp_lastread );
 
+    /* XXX022 this clause may be redundant with the clause in
+     * connection_ap_handshake_attach_circuit(). Is it? -RD */
     if (AP_CONN_STATE_IS_UNATTACHED(conn->_base.state)) {
       if (seconds_idle >= options->SocksTimeout) {
         log_fn(severity, LD_APP,