DW:

 - Disabling a mysterious block at the beginning of httpRequestFree
   that causes incomplete transfers to be logged with status code
   "000."  There seems to be no bad side effects from taking it out.

diff --git a/src/client_side.cc b/src/client_side.cc
index 7e5fed0b423618e78df5b7f94ad7294b650486fe..d7e3f4512e83a9f6e24dfebc0d3a52e2362242ea 100644 (file)
--- a/src/client_side.cc
+++ b/src/client_side.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: client_side.cc,v 1.474 2000/03/28 17:41:39 wessels Exp $
+ * $Id: client_side.cc,v 1.475 2000/04/18 03:20:26 wessels Exp $
  *
  * DEBUG: section 33    Client-side Routines
  * AUTHOR: Duane Wessels
@@ -645,11 +645,18 @@ httpRequestFree(void *data)
     MemObject *mem = NULL;
     debug(33, 3) ("httpRequestFree: %s\n", storeUrl(http->entry));
     if (!clientCheckTransferDone(http)) {
+#if MYSTERIOUS_CODE
+       /*
+        * DW: this seems odd here, is it really needed?  It causes
+        * incomplete transfers to get logged with "000" status
+        * code because http->entry becomes NULL.
+        */
        if ((e = http->entry)) {
            http->entry = NULL;
            storeUnregister(e, http);
            storeUnlockObject(e);
        }
+#endif
        if (http->entry && http->entry->ping_status == PING_WAITING)
            storeReleaseRequest(http->entry);
     }