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.


svn:r18383
This commit is contained in:
Roger Dingledine 2009-02-04 08:27:36 +00:00
parent 893479b8b3
commit 45aeabf8f7

View File

@ -409,6 +409,8 @@ connection_ap_expire_beginning(void)
? LOG_INFO : LOG_NOTICE; ? LOG_INFO : LOG_NOTICE;
seconds_idle = (int)( now - conn->_base.timestamp_lastread ); 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 (AP_CONN_STATE_IS_UNATTACHED(conn->_base.state)) {
if (seconds_idle >= options->SocksTimeout) { if (seconds_idle >= options->SocksTimeout) {
log_fn(severity, LD_APP, log_fn(severity, LD_APP,