]> granicus.if.org Git - curl/commitdiff
removed unnecessary check from add_buffer_send() that only was made within
authorDaniel Stenberg <daniel@haxx.se>
Thu, 15 Nov 2007 23:30:59 +0000 (23:30 +0000)
committerDaniel Stenberg <daniel@haxx.se>
Thu, 15 Nov 2007 23:30:59 +0000 (23:30 +0000)
#ifdef CURL_DOES_CONVERSIONS anyway! I turned it into a DEBUGASSERT() instead.

lib/http.c

index e7b39ad4bb696fa63b1206e9d307a146259de3e6..eba69bdaceea57aad88d1c8d1e01389ff27bc96d 100644 (file)
@@ -943,17 +943,17 @@ CURLcode add_buffer_send(send_buffer *in,
   ptr = in->buffer;
   size = in->size_used;
 
+  DEBUGASSERT(size - included_body_bytes > 0);
+
 #ifdef CURL_DOES_CONVERSIONS
-  if(size - included_body_bytes > 0) {
-    res = Curl_convert_to_network(conn->data, ptr, size - included_body_bytes);
-    /* Curl_convert_to_network calls failf if unsuccessful */
-    if(res != CURLE_OK) {
-      /* conversion failed, free memory and return to the caller */
-      if(in->buffer)
-        free(in->buffer);
-      free(in);
-      return res;
-    }
+  res = Curl_convert_to_network(conn->data, ptr, size - included_body_bytes);
+  /* Curl_convert_to_network calls failf if unsuccessful */
+  if(res != CURLE_OK) {
+    /* conversion failed, free memory and return to the caller */
+    if(in->buffer)
+      free(in->buffer);
+    free(in);
+    return res;
   }
 #endif /* CURL_DOES_CONVERSIONS */