]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
libtests: update format strings to avoid casts, drop some macros
authorViktor Szakats <commit@vsz.me>
Wed, 30 Jul 2025 00:12:22 +0000 (02:12 +0200)
committerViktor Szakats <commit@vsz.me>
Thu, 31 Jul 2025 07:29:49 +0000 (09:29 +0200)
- bump format strings to show the full value, drop casts.
- drop redundant casts (enum -> `%d`).
- drop some single-use macros.
- replace `int` with `bool` in testtrace.

Closes #18106

63 files changed:
tests/libtest/cli_h2_pausing.c
tests/libtest/cli_h2_serverpush.c
tests/libtest/cli_hx_download.c
tests/libtest/cli_hx_upload.c
tests/libtest/cli_ws_data.c
tests/libtest/cli_ws_pingpong.c
tests/libtest/first.h
tests/libtest/lib1485.c
tests/libtest/lib1506.c
tests/libtest/lib1507.c
tests/libtest/lib1512.c
tests/libtest/lib1515.c
tests/libtest/lib1520.c
tests/libtest/lib1522.c
tests/libtest/lib1538.c
tests/libtest/lib1540.c
tests/libtest/lib1542.c
tests/libtest/lib1553.c
tests/libtest/lib1556.c
tests/libtest/lib1559.c
tests/libtest/lib1560.c
tests/libtest/lib1565.c
tests/libtest/lib1907.c
tests/libtest/lib1911.c
tests/libtest/lib1915.c
tests/libtest/lib1940.c
tests/libtest/lib1945.c
tests/libtest/lib2032.c
tests/libtest/lib2301.c
tests/libtest/lib2402.c
tests/libtest/lib2404.c
tests/libtest/lib2502.c
tests/libtest/lib2700.c
tests/libtest/lib3026.c
tests/libtest/lib3033.c
tests/libtest/lib3105.c
tests/libtest/lib500.c
tests/libtest/lib506.c
tests/libtest/lib526.c
tests/libtest/lib530.c
tests/libtest/lib536.c
tests/libtest/lib540.c
tests/libtest/lib544.c
tests/libtest/lib547.c
tests/libtest/lib552.c
tests/libtest/lib554.c
tests/libtest/lib555.c
tests/libtest/lib557.c
tests/libtest/lib571.c
tests/libtest/lib573.c
tests/libtest/lib574.c
tests/libtest/lib576.c
tests/libtest/lib582.c
tests/libtest/lib583.c
tests/libtest/lib586.c
tests/libtest/lib650.c
tests/libtest/lib651.c
tests/libtest/lib670.c
tests/libtest/lib677.c
tests/libtest/lib753.c
tests/libtest/testtrace.c
tests/libtest/testtrace.h
tests/unit/unit1309.c

index ecdf8f1596fcfe12cea819f0a1b2484b24bdd297..8cc440daacfc83ea707696de8bde65cf5761b0ae 100644 (file)
@@ -58,25 +58,25 @@ static size_t cb(char *data, size_t size, size_t nmemb, void *clientp)
   (void)data;
   if(curl_easy_getinfo(handle->h, CURLINFO_CONTENT_LENGTH_DOWNLOAD_T,
                        &totalsize) == CURLE_OK)
-    curl_mfprintf(stderr, "INFO: [%d] write, "
+    curl_mfprintf(stderr, "INFO: [%zu] write, "
                   "Content-Length %" CURL_FORMAT_CURL_OFF_T "\n",
-                  (int)handle->idx, totalsize);
+                  handle->idx, totalsize);
 
   if(!handle->resumed) {
     ++handle->paused;
-    curl_mfprintf(stderr, "INFO: [%d] write, PAUSING %d time on %lu bytes\n",
-                  (int)handle->idx, handle->paused, (long)realsize);
+    curl_mfprintf(stderr, "INFO: [%zu] write, PAUSING %d time on %zu bytes\n",
+                  handle->idx, handle->paused, realsize);
     assert(handle->paused == 1);
     return CURL_WRITEFUNC_PAUSE;
   }
   if(handle->fail_write) {
     ++handle->errored;
-    curl_mfprintf(stderr, "INFO: [%d] FAIL write of %lu bytes, %d time\n",
-                  (int)handle->idx, (long)realsize, handle->errored);
+    curl_mfprintf(stderr, "INFO: [%zu] FAIL write of %zu bytes, %d time\n",
+                  handle->idx, realsize, handle->errored);
     return CURL_WRITEFUNC_ERROR;
   }
-  curl_mfprintf(stderr, "INFO: [%d] write, accepting %lu bytes\n",
-                (int)handle->idx, (long)realsize);
+  curl_mfprintf(stderr, "INFO: [%zu] write, accepting %zu bytes\n",
+                handle->idx, realsize);
   return realsize;
 }
 
@@ -207,21 +207,21 @@ static CURLcode test_cli_h2_pausing(const char *URL)
       curl_mfprintf(stderr, "INFO: no more handles running\n");
       for(i = 0; i < CURL_ARRAYSIZE(handles); i++) {
         if(!handles[i].paused) {
-          curl_mfprintf(stderr, "ERROR: [%d] NOT PAUSED\n", (int)i);
+          curl_mfprintf(stderr, "ERROR: [%zu] NOT PAUSED\n", i);
           as_expected = 0;
         }
         else if(handles[i].paused != 1) {
-          curl_mfprintf(stderr, "ERROR: [%d] PAUSED %d times!\n",
-                        (int)i, handles[i].paused);
+          curl_mfprintf(stderr, "ERROR: [%zu] PAUSED %d times!\n",
+                        i, handles[i].paused);
           as_expected = 0;
         }
         else if(!handles[i].resumed) {
-          curl_mfprintf(stderr, "ERROR: [%d] NOT resumed!\n", (int)i);
+          curl_mfprintf(stderr, "ERROR: [%zu] NOT resumed!\n", i);
           as_expected = 0;
         }
         else if(handles[i].errored != 1) {
-          curl_mfprintf(stderr, "ERROR: [%d] NOT errored once, %d instead!\n",
-                        (int)i, handles[i].errored);
+          curl_mfprintf(stderr, "ERROR: [%zu] NOT errored once, %d instead!\n",
+                        i, handles[i].errored);
           as_expected = 0;
         }
       }
@@ -242,8 +242,9 @@ static CURLcode test_cli_h2_pausing(const char *URL)
         for(i = 0; i < CURL_ARRAYSIZE(handles); i++) {
           if(msg->easy_handle == handles[i].h) {
             if(handles[i].paused != 1 || !handles[i].resumed) {
-              curl_mfprintf(stderr, "ERROR: [%d] done, pauses=%d, resumed=%d, "
-                            "result %d - wtf?\n", (int)i, handles[i].paused,
+              curl_mfprintf(stderr, "ERROR: [%zu] done, paused=%d, "
+                            "resumed=%d, result %d - wtf?\n", i,
+                            handles[i].paused,
                             handles[i].resumed, msg->data.result);
               rc = (CURLcode)1;
               goto out;
@@ -270,7 +271,7 @@ static CURLcode test_cli_h2_pausing(const char *URL)
     if(resume_round > 0 && rounds == resume_round) {
       /* time to resume */
       for(i = 0; i < CURL_ARRAYSIZE(handles); i++) {
-        curl_mfprintf(stderr, "INFO: [%d] resumed\n", (int)i);
+        curl_mfprintf(stderr, "INFO: [%zu] resumed\n", i);
         handles[i].resumed = 1;
         curl_easy_pause(handles[i].h, CURLPAUSE_CONT);
       }
index ace7fed440bbdf75e3fafabb5a25c61e3a23a22b..a58f38dd0a68e1068084a968c2d8273606d1782d 100644 (file)
@@ -84,11 +84,11 @@ static int server_push_callback(CURL *parent,
   curl_easy_setopt(easy, CURLOPT_WRITEDATA, out_push);
 
   curl_mfprintf(stderr, "**** push callback approves stream %u, "
-                "got %lu headers!\n", count, (unsigned long)num_headers);
+                "got %zu headers!\n", count, num_headers);
 
   for(i = 0; i < num_headers; i++) {
     headp = curl_pushheader_bynum(headers, i);
-    curl_mfprintf(stderr, "**** header %lu: %s\n", (unsigned long)i, headp);
+    curl_mfprintf(stderr, "**** header %zu: %s\n", i, headp);
   }
 
   headp = curl_pushheader_byname(headers, ":path");
@@ -114,8 +114,8 @@ static CURLcode test_cli_h2_serverpush(const char *URL)
   int transfers = 1; /* we start with one */
   struct CURLMsg *m;
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 0;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = FALSE;
 
   if(!URL) {
     curl_mfprintf(stderr, "need URL as argument\n");
index 66ca0ac219dd127d457e9a5ad5626768a7f55d0b..905f431288773110376ff188c737b6909c5adee8 100644 (file)
@@ -29,7 +29,7 @@
 static int verbose_d = 1;
 
 struct transfer_d {
-  int idx;
+  size_t idx;
   CURL *easy;
   char filename[128];
   FILE *out;
@@ -65,10 +65,12 @@ static size_t my_write_d_cb(char *buf, size_t nitems, size_t buflen,
   size_t blen = (nitems * buflen);
   size_t nwritten;
 
-  curl_mfprintf(stderr, "[t-%d] RECV %ld bytes, total=%ld, pause_at=%ld\n",
-                t->idx, (long)blen, (long)t->recv_size, (long)t->pause_at);
+  curl_mfprintf(stderr, "[t-%zu] RECV %zu bytes, "
+                "total=%" CURL_FORMAT_CURL_OFF_T ", "
+                "pause_at=%" CURL_FORMAT_CURL_OFF_T "\n",
+                t->idx, blen, t->recv_size, t->pause_at);
   if(!t->out) {
-    curl_msnprintf(t->filename, sizeof(t->filename)-1, "download_%u.data",
+    curl_msnprintf(t->filename, sizeof(t->filename)-1, "download_%zu.data",
                    t->idx);
     t->out = fopen(t->filename, "wb");
     if(!t->out)
@@ -78,20 +80,20 @@ static size_t my_write_d_cb(char *buf, size_t nitems, size_t buflen,
   if(!t->resumed &&
      t->recv_size < t->pause_at &&
      ((t->recv_size + (curl_off_t)blen) >= t->pause_at)) {
-    curl_mfprintf(stderr, "[t-%d] PAUSE\n", t->idx);
+    curl_mfprintf(stderr, "[t-%zu] PAUSE\n", t->idx);
     t->paused = 1;
     return CURL_WRITEFUNC_PAUSE;
   }
 
   nwritten = fwrite(buf, nitems, buflen, t->out);
   if(nwritten < blen) {
-    curl_mfprintf(stderr, "[t-%d] write failure\n", t->idx);
+    curl_mfprintf(stderr, "[t-%zu] write failure\n", t->idx);
     return 0;
   }
   t->recv_size += (curl_off_t)nwritten;
   if(t->fail_at > 0 && t->recv_size >= t->fail_at) {
-    curl_mfprintf(stderr, "[t-%d] FAIL by write callback at %ld bytes\n",
-                  t->idx, (long)t->recv_size);
+    curl_mfprintf(stderr, "[t-%zu] FAIL by write callback at "
+                  "%" CURL_FORMAT_CURL_OFF_T " bytes\n", t->idx, t->recv_size);
     return CURL_WRITEFUNC_ERROR;
   }
 
@@ -107,8 +109,8 @@ static int my_progress_d_cb(void *userdata,
   (void)ulnow;
   (void)dltotal;
   if(t->abort_at > 0 && dlnow >= t->abort_at) {
-    curl_mfprintf(stderr, "[t-%d] ABORT by progress_cb at %ld bytes\n",
-                  t->idx, (long)dlnow);
+    curl_mfprintf(stderr, "[t-%zu] ABORT by progress_cb at "
+                  "%" CURL_FORMAT_CURL_OFF_T " bytes\n", t->idx, dlnow);
     return 1;
   }
   return 0;
@@ -312,7 +314,7 @@ static CURLcode test_cli_hx_download(const char *URL)
   active_transfers = 0;
   for(i = 0; i < transfer_count_d; ++i) {
     t = &transfer_d[i];
-    t->idx = (int)i;
+    t->idx = i;
     t->abort_at = (curl_off_t)abort_offset;
     t->fail_at = (curl_off_t)fail_offset;
     t->pause_at = (curl_off_t)pause_offset;
@@ -325,14 +327,14 @@ static CURLcode test_cli_hx_download(const char *URL)
     if(!t->easy ||
        setup_hx_download(t->easy, url, t, http_version, host, share,
                          use_earlydata, fresh_connect)) {
-      curl_mfprintf(stderr, "[t-%d] FAILED setup\n", (int)i);
+      curl_mfprintf(stderr, "[t-%zu] FAILED setup\n", i);
       result = (CURLcode)1;
       goto cleanup;
     }
     curl_multi_add_handle(multi_handle, t->easy);
     t->started = 1;
     ++active_transfers;
-    curl_mfprintf(stderr, "[t-%d] STARTED\n", t->idx);
+    curl_mfprintf(stderr, "[t-%zu] STARTED\n", t->idx);
   }
 
   do {
@@ -358,13 +360,13 @@ static CURLcode test_cli_hx_download(const char *URL)
         if(t) {
           t->done = 1;
           t->result = m->data.result;
-          curl_mfprintf(stderr, "[t-%d] FINISHED with result %d\n",
+          curl_mfprintf(stderr, "[t-%zu] FINISHED with result %d\n",
                         t->idx, t->result);
           if(use_earlydata) {
             curl_off_t sent;
             curl_easy_getinfo(e, CURLINFO_EARLYDATA_SENT_T, &sent);
-            curl_mfprintf(stderr, "[t-%d] EarlyData: %ld\n", t->idx,
-                          (long)sent);
+            curl_mfprintf(stderr, "[t-%zu] EarlyData: "
+                          "%" CURL_FORMAT_CURL_OFF_T "\n", t->idx, sent);
           }
         }
         else {
@@ -382,7 +384,7 @@ static CURLcode test_cli_hx_download(const char *URL)
             curl_multi_remove_handle(multi_handle, t->easy);
             t->done = 1;
             active_transfers--;
-            curl_mfprintf(stderr, "[t-%d] ABORTED\n", t->idx);
+            curl_mfprintf(stderr, "[t-%zu] ABORTED\n", t->idx);
           }
         }
       }
@@ -394,7 +396,7 @@ static CURLcode test_cli_hx_download(const char *URL)
             t->resumed = 1;
             t->paused = 0;
             curl_easy_pause(t->easy, CURLPAUSE_CONT);
-            curl_mfprintf(stderr, "[t-%d] RESUMED\n", t->idx);
+            curl_mfprintf(stderr, "[t-%zu] RESUMED\n", t->idx);
             break;
           }
         }
@@ -408,14 +410,14 @@ static CURLcode test_cli_hx_download(const char *URL)
             if(!t->easy ||
               setup_hx_download(t->easy, url, t, http_version, host, share,
                                 use_earlydata, fresh_connect)) {
-              curl_mfprintf(stderr, "[t-%d] FAILED setup\n", (int)i);
+              curl_mfprintf(stderr, "[t-%zu] FAILED setup\n", i);
               result = (CURLcode)1;
               goto cleanup;
             }
             curl_multi_add_handle(multi_handle, t->easy);
             t->started = 1;
             ++active_transfers;
-            curl_mfprintf(stderr, "[t-%d] STARTED\n", t->idx);
+            curl_mfprintf(stderr, "[t-%zu] STARTED\n", t->idx);
             break;
           }
         }
index b05694b4d4b10b6098e4bc3b3045f32195ed8c1e..9c69e36787f4e2ebd28ff12dd7f89ee3ae4f88e4 100644 (file)
@@ -29,7 +29,7 @@
 static int verbose_u = 1;
 
 struct transfer_u {
-  int idx;
+  size_t idx;
   CURL *easy;
   const char *method;
   char filename[128];
@@ -67,10 +67,12 @@ static size_t my_write_u_cb(char *buf, size_t nitems, size_t buflen,
   size_t blen = (nitems * buflen);
   size_t nwritten;
 
-  curl_mfprintf(stderr, "[t-%d] RECV %ld bytes, total=%ld, pause_at=%ld\n",
-                t->idx, (long)blen, (long)t->recv_size, (long)t->pause_at);
+  curl_mfprintf(stderr, "[t-%zu] RECV %zu bytes, "
+                "total=%" CURL_FORMAT_CURL_OFF_T ", "
+                "pause_at=%" CURL_FORMAT_CURL_OFF_T "\n",
+                t->idx, blen, t->recv_size, t->pause_at);
   if(!t->out) {
-    curl_msnprintf(t->filename, sizeof(t->filename)-1, "download_%u.data",
+    curl_msnprintf(t->filename, sizeof(t->filename)-1, "download_%zu.data",
                    t->idx);
     t->out = fopen(t->filename, "wb");
     if(!t->out)
@@ -79,7 +81,7 @@ static size_t my_write_u_cb(char *buf, size_t nitems, size_t buflen,
 
   nwritten = fwrite(buf, nitems, buflen, t->out);
   if(nwritten < blen) {
-    curl_mfprintf(stderr, "[t-%d] write failure\n", t->idx);
+    curl_mfprintf(stderr, "[t-%zu] write failure\n", t->idx);
     return 0;
   }
   t->recv_size += (curl_off_t)nwritten;
@@ -100,13 +102,15 @@ static size_t my_read_cb(char *buf, size_t nitems, size_t buflen,
   else
     nread = blen;
 
-  curl_mfprintf(stderr, "[t-%d] SEND %ld bytes, total=%ld, pause_at=%ld\n",
-                t->idx, (long)nread, (long)t->send_total, (long)t->pause_at);
+  curl_mfprintf(stderr, "[t-%zu] SEND %zu bytes, "
+                "total=%" CURL_FORMAT_CURL_OFF_T ", "
+                "pause_at=%" CURL_FORMAT_CURL_OFF_T "\n",
+                t->idx, nread, t->send_total, t->pause_at);
 
   if(!t->resumed &&
      t->send_size < t->pause_at &&
      ((t->send_size + (curl_off_t)blen) >= t->pause_at)) {
-    curl_mfprintf(stderr, "[t-%d] PAUSE\n", t->idx);
+    curl_mfprintf(stderr, "[t-%zu] PAUSE\n", t->idx);
     t->paused = 1;
     return CURL_READFUNC_PAUSE;
   }
@@ -114,8 +118,8 @@ static size_t my_read_cb(char *buf, size_t nitems, size_t buflen,
   memset(buf, 'x', nread);
   t->send_size += (curl_off_t)nread;
   if(t->fail_at > 0 && t->send_size >= t->fail_at) {
-    curl_mfprintf(stderr, "[t-%d] ABORT by read callback at %ld bytes\n",
-                  t->idx, (long)t->send_size);
+    curl_mfprintf(stderr, "[t-%zu] ABORT by read callback at "
+                  "%" CURL_FORMAT_CURL_OFF_T " bytes\n", t->idx, t->send_size);
     return CURL_READFUNC_ABORT;
   }
   return (size_t)nread;
@@ -130,8 +134,8 @@ static int my_progress_u_cb(void *userdata,
   (void)dlnow;
   (void)dltotal;
   if(t->abort_at > 0 && ulnow >= t->abort_at) {
-    curl_mfprintf(stderr, "[t-%d] ABORT by progress_cb at %ld bytes sent\n",
-                  t->idx, (long)ulnow);
+    curl_mfprintf(stderr, "[t-%zu] ABORT by progress_cb at "
+                  "%" CURL_FORMAT_CURL_OFF_T " bytes sent\n", t->idx, ulnow);
     return 1;
   }
   return 0;
@@ -338,7 +342,7 @@ static CURLcode test_cli_hx_upload(const char *URL)
   active_transfers = 0;
   for(i = 0; i < transfer_count_u; ++i) {
     t = &transfer_u[i];
-    t->idx = (int)i;
+    t->idx = i;
     t->method = method;
     t->send_total = (curl_off_t)send_total;
     t->abort_at = (curl_off_t)abort_offset;
@@ -358,13 +362,13 @@ static CURLcode test_cli_hx_upload(const char *URL)
       t->easy = easy;
       if(setup_hx_upload(t->easy, url, t, http_version, host, share,
                          use_earlydata, announce_length)) {
-        curl_mfprintf(stderr, "[t-%d] FAILED setup\n", (int)i);
+        curl_mfprintf(stderr, "[t-%zu] FAILED setup\n", i);
         return (CURLcode)1;
       }
 
-      curl_mfprintf(stderr, "[t-%d] STARTING\n", t->idx);
+      curl_mfprintf(stderr, "[t-%zu] STARTING\n", t->idx);
       rc = curl_easy_perform(easy);
-      curl_mfprintf(stderr, "[t-%d] DONE -> %d\n", t->idx, rc);
+      curl_mfprintf(stderr, "[t-%zu] DONE -> %d\n", t->idx, rc);
       t->easy = NULL;
       curl_easy_reset(easy);
     }
@@ -380,13 +384,13 @@ static CURLcode test_cli_hx_upload(const char *URL)
       t->easy = curl_easy_init();
       if(!t->easy || setup_hx_upload(t->easy, url, t, http_version, host,
                                      share, use_earlydata, announce_length)) {
-        curl_mfprintf(stderr, "[t-%d] FAILED setup\n", (int)i);
+        curl_mfprintf(stderr, "[t-%zu] FAILED setup\n", i);
         return (CURLcode)1;
       }
       curl_multi_add_handle(multi_handle, t->easy);
       t->started = 1;
       ++active_transfers;
-      curl_mfprintf(stderr, "[t-%d] STARTED\n", t->idx);
+      curl_mfprintf(stderr, "[t-%zu] STARTED\n", t->idx);
     }
 
     do {
@@ -414,13 +418,14 @@ static CURLcode test_cli_hx_upload(const char *URL)
             long res_status;
             curl_easy_getinfo(e, CURLINFO_RESPONSE_CODE, &res_status);
             t->done = 1;
-            curl_mfprintf(stderr, "[t-%d] FINISHED, result=%d, response=%ld\n",
+            curl_mfprintf(stderr, "[t-%zu] FINISHED, "
+                          "result=%d, response=%ld\n",
                           t->idx, m->data.result, res_status);
             if(use_earlydata) {
               curl_off_t sent;
               curl_easy_getinfo(e, CURLINFO_EARLYDATA_SENT_T, &sent);
-              curl_mfprintf(stderr, "[t-%d] EarlyData: %ld\n", t->idx,
-                            (long)sent);
+              curl_mfprintf(stderr, "[t-%zu] EarlyData: "
+                            "%" CURL_FORMAT_CURL_OFF_T "\n", t->idx, sent);
             }
           }
           else {
@@ -439,7 +444,7 @@ static CURLcode test_cli_hx_upload(const char *URL)
               curl_multi_remove_handle(multi_handle, t->easy);
               t->done = 1;
               active_transfers--;
-              curl_mfprintf(stderr, "[t-%d] ABORTED\n", t->idx);
+              curl_mfprintf(stderr, "[t-%zu] ABORTED\n", t->idx);
             }
           }
         }
@@ -451,7 +456,7 @@ static CURLcode test_cli_hx_upload(const char *URL)
               t->resumed = 1;
               t->paused = 0;
               curl_easy_pause(t->easy, CURLPAUSE_CONT);
-              curl_mfprintf(stderr, "[t-%d] RESUMED\n", t->idx);
+              curl_mfprintf(stderr, "[t-%zu] RESUMED\n", t->idx);
               break;
             }
           }
@@ -465,13 +470,13 @@ static CURLcode test_cli_hx_upload(const char *URL)
               if(!t->easy || setup_hx_upload(t->easy, url, t, http_version,
                                              host, share, use_earlydata,
                                              announce_length)) {
-                curl_mfprintf(stderr, "[t-%d] FAILED setup\n", (int)i);
+                curl_mfprintf(stderr, "[t-%zu] FAILED setup\n", i);
                 return (CURLcode)1;
               }
               curl_multi_add_handle(multi_handle, t->easy);
               t->started = 1;
               ++active_transfers;
-              curl_mfprintf(stderr, "[t-%d] STARTED\n", t->idx);
+              curl_mfprintf(stderr, "[t-%zu] STARTED\n", t->idx);
               break;
             }
           }
index 2c659a385773706b0349795f491185377af4a6b3..82717415b9fc884742016a7ba6b21139e70976aa 100644 (file)
@@ -36,30 +36,32 @@ static CURLcode check_recv(const struct curl_ws_frame *frame,
 
   if(frame->flags & CURLWS_CLOSE) {
     curl_mfprintf(stderr, "recv_data: unexpected CLOSE frame from server, "
-                  "got %ld bytes, offset=%ld, rflags %x\n",
-                  (long)nread, (long)r_offset, frame->flags);
+                  "got %zu bytes, offset=%zu, rflags %x\n",
+                  nread, r_offset, frame->flags);
     return CURLE_RECV_ERROR;
   }
   if(!r_offset && !(frame->flags & CURLWS_BINARY)) {
-    curl_mfprintf(stderr, "recv_data: wrong frame, got %ld bytes, offset=%ld, "
+    curl_mfprintf(stderr, "recv_data: wrong frame, got %zu bytes, offset=%zu, "
                   "rflags %x\n",
-                  (long)nread, (long)r_offset, frame->flags);
+                  nread, r_offset, frame->flags);
     return CURLE_RECV_ERROR;
   }
   if(frame->offset != (curl_off_t)r_offset) {
-    curl_mfprintf(stderr, "recv_data: frame offset, expected %ld, got %ld\n",
-                  (long)r_offset, (long)frame->offset);
+    curl_mfprintf(stderr, "recv_data: frame offset, expected %zu, "
+                  "got %" CURL_FORMAT_CURL_OFF_T "\n",
+                  r_offset, frame->offset);
     return CURLE_RECV_ERROR;
   }
   if(frame->bytesleft != (curl_off_t)(exp_len - r_offset - nread)) {
     curl_mfprintf(stderr, "recv_data: frame bytesleft, "
-                  "expected %ld, got %ld\n",
-                  (long)(exp_len - r_offset - nread), (long)frame->bytesleft);
+                  "expected %" CURL_FORMAT_CURL_OFF_T ", "
+                  "got %" CURL_FORMAT_CURL_OFF_T "\n",
+                  (curl_off_t)(exp_len - r_offset - nread), frame->bytesleft);
     return CURLE_RECV_ERROR;
   }
   if(r_offset + nread > exp_len) {
-    curl_mfprintf(stderr, "recv_data: data length, expected %ld, now at %ld\n",
-                  (long)exp_len, (long)(r_offset + nread));
+    curl_mfprintf(stderr, "recv_data: data length, expected %zu, now at %zu\n",
+                  exp_len, r_offset + nread);
     return CURLE_RECV_ERROR;
   }
   return CURLE_OK;
@@ -97,9 +99,9 @@ static CURLcode data_echo(CURL *curl, size_t count,
         r = curl_ws_send(curl, sbuf, slen, &nwritten, 0, CURLWS_BINARY);
         sblock = (r == CURLE_AGAIN);
         if(!r || (r == CURLE_AGAIN)) {
-          curl_mfprintf(stderr, "curl_ws_send(len=%ld) -> %d, %ld (%ld/%ld)\n",
-                        (long)slen, r, (long)nwritten,
-                        (long)(len - slen), (long)len);
+          curl_mfprintf(stderr, "curl_ws_send(len=%zu) -> %d, "
+                        "%zu (%" CURL_FORMAT_CURL_OFF_T "/%zu)\n",
+                        slen, r, nwritten, (curl_off_t)(len - slen), len);
           sbuf += nwritten;
           slen -= nwritten;
         }
@@ -118,10 +120,8 @@ static CURLcode data_echo(CURL *curl, size_t count,
                          &nread, &frame);
         if(!r || (r == CURLE_AGAIN)) {
           rblock = (r == CURLE_AGAIN);
-          curl_mfprintf(stderr, "curl_ws_recv(len=%ld) -> %d, %ld (%ld/%ld) "
-                        "\n",
-                        (long)rlen, r, (long)nread, (long)(len - rlen),
-                        (long)len);
+          curl_mfprintf(stderr, "curl_ws_recv(len=%zu) -> %d, %zu (%ld/%zu) "
+                        "\n", rlen, r, nread, (long)(len - rlen), len);
           if(!r) {
             r = check_recv(frame, len - rlen, nread, len);
             if(r)
@@ -148,9 +148,9 @@ static CURLcode data_echo(CURL *curl, size_t count,
     if(memcmp(send_buf, recv_buf, len)) {
       curl_mfprintf(stderr, "recv_data: data differs\n");
       debug_dump("", "expected:", stderr,
-                 (const unsigned char *)send_buf, len, 0);
+                 (const unsigned char *)send_buf, len, FALSE);
       debug_dump("", "received:", stderr,
-                 (const unsigned char *)recv_buf, len, 0);
+                 (const unsigned char *)recv_buf, len, FALSE);
       r = CURLE_RECV_ERROR;
       goto out;
     }
@@ -216,8 +216,8 @@ static CURLcode test_cli_ws_data(const char *URL)
     plen_max = plen_min;
 
   if(plen_max < plen_min) {
-    curl_mfprintf(stderr, "maxlen must be >= minlen, got %ld-%ld\n",
-                  (long)plen_min, (long)plen_max);
+    curl_mfprintf(stderr, "maxlen must be >= minlen, got %zu-%zu\n",
+                  plen_min, plen_max);
     res = CURLE_BAD_FUNCTION_ARGUMENT;
     goto cleanup;
   }
@@ -240,7 +240,7 @@ static CURLcode test_cli_ws_data(const char *URL)
     curl_easy_setopt(curl, CURLOPT_VERBOSE, 1L);
     curl_easy_setopt(curl, CURLOPT_CONNECT_ONLY, 2L); /* websocket style */
     res = curl_easy_perform(curl);
-    curl_mfprintf(stderr, "curl_easy_perform() returned %u\n", (int)res);
+    curl_mfprintf(stderr, "curl_easy_perform() returned %u\n", res);
     if(res == CURLE_OK)
       res = data_echo(curl, count, plen_min, plen_max);
 
index fc96bbf2d302c429284dfccf65c8c80171f281c1..80707c95852118af2c8589e446316a447e91fa9d 100644 (file)
@@ -76,7 +76,7 @@ static CURLcode test_cli_ws_pingpong(const char *URL)
     curl_easy_setopt(curl, CURLOPT_VERBOSE, 1L);
     curl_easy_setopt(curl, CURLOPT_CONNECT_ONLY, 2L); /* websocket style */
     res = curl_easy_perform(curl);
-    curl_mfprintf(stderr, "curl_easy_perform() returned %u\n", (int)res);
+    curl_mfprintf(stderr, "curl_easy_perform() returned %u\n", res);
     if(res == CURLE_OK)
       res = pingpong(curl, payload);
 
index 32af7fe62d6567e0af9eaa1f736e5e639f463a96..1592922e39a88bdc56472e7c635b83e1a1afa933 100644 (file)
@@ -193,7 +193,7 @@ void ws_close(CURL *curl);
   if((ec = curl_easy_setopt((A), (B), (C))) != CURLE_OK) {    \
     curl_mfprintf(stderr, "%s:%d curl_easy_setopt() failed, " \
                   "with code %d (%s)\n",                      \
-                  (Y), (Z), (int)ec, curl_easy_strerror(ec)); \
+                  (Y), (Z), ec, curl_easy_strerror(ec));      \
     res = ec;                                                 \
   }                                                           \
 } while(0)
@@ -217,7 +217,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_setopt((A), (B), (C))) != CURLM_OK) {    \
     curl_mfprintf(stderr, "%s:%d curl_multi_setopt() failed, " \
                   "with code %d (%s)\n",                       \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec)); \
+                  (Y), (Z), ec, curl_multi_strerror(ec));      \
     res = TEST_ERR_MULTI;                                      \
   }                                                            \
 } while(0)
@@ -241,7 +241,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_add_handle((A), (B))) != CURLM_OK) {         \
     curl_mfprintf(stderr, "%s:%d curl_multi_add_handle() failed, " \
                   "with code %d (%s)\n",                           \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec));     \
+                  (Y), (Z), ec, curl_multi_strerror(ec));          \
     res = TEST_ERR_MULTI;                                          \
   }                                                                \
 } while(0)
@@ -265,7 +265,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_remove_handle((A), (B))) != CURLM_OK) {         \
     curl_mfprintf(stderr, "%s:%d curl_multi_remove_handle() failed, " \
                   "with code %d (%s)\n",                              \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec));        \
+                  (Y), (Z), ec, curl_multi_strerror(ec));             \
     res = TEST_ERR_MULTI;                                             \
   }                                                                   \
 } while(0)
@@ -290,7 +290,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_perform((A), (B))) != CURLM_OK) {                \
     curl_mfprintf(stderr, "%s:%d curl_multi_perform() failed, "        \
                   "with code %d (%s)\n",                               \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec));         \
+                  (Y), (Z), ec, curl_multi_strerror(ec));              \
     res = TEST_ERR_MULTI;                                              \
   }                                                                    \
   else if(*((B)) < 0) {                                                \
@@ -320,7 +320,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_fdset((A), (B), (C), (D), (E))) != CURLM_OK) { \
     curl_mfprintf(stderr, "%s:%d curl_multi_fdset() failed, "        \
                   "with code %d (%s)\n",                             \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec));       \
+                  (Y), (Z), ec, curl_multi_strerror(ec));            \
     res = TEST_ERR_MULTI;                                            \
   }                                                                  \
   else if(*((E)) < -1) {                                             \
@@ -350,7 +350,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_timeout((A), (B))) != CURLM_OK) {            \
     curl_mfprintf(stderr, "%s:%d curl_multi_timeout() failed, "    \
                   "with code %d (%s)\n",                           \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec));     \
+                  (Y), (Z), ec, curl_multi_strerror(ec));          \
     res = TEST_ERR_BAD_TIMEOUT;                                    \
   }                                                                \
   else if(*((B)) < -1L) {                                          \
@@ -380,7 +380,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_poll((A), (B), (C), (D), (E))) != CURLM_OK) { \
     curl_mfprintf(stderr, "%s:%d curl_multi_poll() failed, "        \
                   "with code %d (%s)\n",                            \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec));      \
+                  (Y), (Z), ec, curl_multi_strerror(ec));           \
     res = TEST_ERR_MULTI;                                           \
   }                                                                 \
   else if(*((E)) < 0) {                                             \
@@ -410,7 +410,7 @@ void ws_close(CURL *curl);
   if((ec = curl_multi_wakeup((A))) != CURLM_OK) {              \
     curl_mfprintf(stderr, "%s:%d curl_multi_wakeup() failed, " \
                   "with code %d (%s)\n",                       \
-                  (Y), (Z), (int)ec, curl_multi_strerror(ec)); \
+                  (Y), (Z), ec, curl_multi_strerror(ec));      \
     res = TEST_ERR_MULTI;                                      \
   }                                                            \
 } while(0)
@@ -495,7 +495,7 @@ void ws_close(CURL *curl);
   if((ec = curl_global_init((A))) != CURLE_OK) {              \
     curl_mfprintf(stderr, "%s:%d curl_global_init() failed, " \
                   "with code %d (%s)\n",                      \
-                  (Y), (Z), (int)ec, curl_easy_strerror(ec)); \
+                  (Y), (Z), ec, curl_easy_strerror(ec));      \
     res = ec;                                                 \
   }                                                           \
 } while(0)
index 0ca017e3e73721cc816eef6b67185f97ec8f818a..09591e199e706f6d2bb83fec3d77b597eb41a773 100644 (file)
@@ -68,7 +68,7 @@ static size_t t1485_header_callback(char *ptr, size_t size, size_t nmemb,
       if(clen < 0) {
         curl_mfprintf(stderr,
                       "header_callback, expected known Content-Length, "
-                      "got: %ld\n", (long)clen);
+                      "got: %" CURL_FORMAT_CURL_OFF_T "\n", clen);
         return CURLE_WRITE_ERROR;
       }
     }
index 0ae7ae928ce99302ec9a5471ff9ae0c389e93ea2..66768f7563790d52801fceeffe4a3056c3e3a7a6 100644 (file)
@@ -42,8 +42,8 @@ static CURLcode test_lib1506(const char *URL)
 
   /* Create fake DNS entries for serverX.example.com for all handles */
   for(i = 0; i < CURL_ARRAYSIZE(curl); i++) {
-    curl_msnprintf(dnsentry, sizeof(dnsentry), "server%d.example.com:%s:%s",
-                   (int)i + 1, port, address);
+    curl_msnprintf(dnsentry, sizeof(dnsentry), "server%zu.example.com:%s:%s",
+                   i + 1, port, address);
     curl_mprintf("%s\n", dnsentry);
     slist2 = curl_slist_append(slist, dnsentry);
     if(!slist2) {
@@ -67,8 +67,8 @@ static CURLcode test_lib1506(const char *URL)
     easy_init(curl[i]);
     /* specify target */
     curl_msnprintf(target_url, sizeof(target_url),
-                   "http://server%d.example.com:%s/path/1506%04i",
-                   (int)i + 1, port, (int)i + 1);
+                   "http://server%zu.example.com:%s/path/1506%04zu",
+                   i + 1, port, i + 1);
     target_url[sizeof(target_url) - 1] = '\0';
     easy_setopt(curl[i], CURLOPT_URL, target_url);
     /* go verbose */
index 1ce0aba148edb7073d0102f046820b610887faa7..8d96c6b3009d1f2c5ca867375a16229dde60a24d 100644 (file)
 
 #include "memdebug.h"
 
-/*
- * This is the list of basic details you need to tweak to get things right.
- */
-#define USERNAME "user@example.com"
-#define PASSWORD "123qwerty"
-#define RECIPIENT "<1507-recipient@example.com>"
-#define MAILFROM "<1507-realuser@example.com>"
-
 static size_t t1507_read_cb(char *ptr, size_t size, size_t nmemb, void *userp)
 {
   (void)ptr;
@@ -59,19 +51,19 @@ static CURLcode test_lib1507(const char *URL)
 
   multi_init(mcurl);
 
-  rcpt_list = curl_slist_append(rcpt_list, RECIPIENT);
+  rcpt_list = curl_slist_append(rcpt_list, "<1507-recipient@example.com>");
 #if 0
   /* more addresses can be added here */
   rcpt_list = curl_slist_append(rcpt_list, "<others@example.com>");
 #endif
   curl_easy_setopt(curl, CURLOPT_URL, URL);
 #if 0
-  curl_easy_setopt(curl, CURLOPT_USERNAME, USERNAME);
-  curl_easy_setopt(curl, CURLOPT_PASSWORD, PASSWORD);
+  curl_easy_setopt(curl, CURLOPT_USERNAME, "user@example.com");
+  curl_easy_setopt(curl, CURLOPT_PASSWORD, "123qwerty");
 #endif
   curl_easy_setopt(curl, CURLOPT_UPLOAD, 1L);
   curl_easy_setopt(curl, CURLOPT_READFUNCTION, t1507_read_cb);
-  curl_easy_setopt(curl, CURLOPT_MAIL_FROM, MAILFROM);
+  curl_easy_setopt(curl, CURLOPT_MAIL_FROM, "<1507-realuser@example.com>");
   curl_easy_setopt(curl, CURLOPT_MAIL_RCPT, rcpt_list);
   curl_easy_setopt(curl, CURLOPT_VERBOSE, 1L);
   multi_add_handle(mcurl, curl);
index 589d903d7551c4f791430018d655c8f1b055670e..6e6287dbfc32f98b35950e0263a105c912d9b8f6 100644 (file)
@@ -60,8 +60,8 @@ static CURLcode test_lib1512(const char *URL)
     easy_init(curl[i]);
     /* specify target */
     curl_msnprintf(target_url, sizeof(target_url),
-                   "http://server.example.curl:%s/path/1512%04i",
-                   port, (int)i + 1);
+                   "http://server.example.curl:%s/path/1512%04zu",
+                   port, i + 1);
     target_url[sizeof(target_url) - 1] = '\0';
     easy_setopt(curl[i], CURLOPT_URL, target_url);
     /* go verbose */
index 348f1fd9acf37c949435fb56de2259ec9773dbfc..02c3c99bf80698a5f493ca353c174b5be0fe826d 100644 (file)
@@ -52,8 +52,8 @@ static CURLcode do_one_request(CURLM *m, const char *URL, const char *resolve)
   easy_setopt(curls, CURLOPT_RESOLVE, resolve_list);
   easy_setopt(curls, CURLOPT_DNS_CACHE_TIMEOUT, DNS_TIMEOUT);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
   easy_setopt(curls, CURLOPT_DEBUGDATA, &debug_config);
   easy_setopt(curls, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   easy_setopt(curls, CURLOPT_VERBOSE, 1L);
index 9d6b3b611d507f1b5b174346021a132060a88811..e61727eadc224089e2a36c22615f2e5c2d296a55 100644 (file)
 
 #include "memdebug.h"
 
-/*
- * This is the list of basic details you need to tweak to get things right.
- */
-#define TO "<recipient@example.com>"
-#define FROM "<sender@example.com>"
-
 struct upload_status {
   int lines_read;
 };
@@ -90,16 +84,16 @@ static CURLcode test_lib1520(const char *URL)
     return TEST_ERR_MAJOR_BAD;
   }
 
-  rcpt_list = curl_slist_append(rcpt_list, TO);
-  /* more addresses can be added here
-     rcpt_list = curl_slist_append(rcpt_list, "<others@example.com>");
-  */
-
+  rcpt_list = curl_slist_append(rcpt_list, "<recipient@example.com>");
+#if 0
+  /* more addresses can be added here */
+  rcpt_list = curl_slist_append(rcpt_list, "<others@example.com>");
+#endif
   test_setopt(curl, CURLOPT_URL, URL);
   test_setopt(curl, CURLOPT_UPLOAD, 1L);
   test_setopt(curl, CURLOPT_READFUNCTION, t1520_read_cb);
   test_setopt(curl, CURLOPT_READDATA, &upload_ctx);
-  test_setopt(curl, CURLOPT_MAIL_FROM, FROM);
+  test_setopt(curl, CURLOPT_MAIL_FROM, "<sender@example.com>");
   test_setopt(curl, CURLOPT_MAIL_RCPT, rcpt_list);
   test_setopt(curl, CURLOPT_VERBOSE, 1L);
 
index 85a901f2ace9d0f46538d2ed956be9a09029c576..3423f4777c959c6bc3b4b7b9722d42afb07f7ff9 100644 (file)
@@ -60,8 +60,8 @@ static CURLcode test_lib1522(const char *URL)
   curl_easy_setopt(curl, CURLOPT_POSTFIELDS, g_Data);
   curl_easy_setopt(curl, CURLOPT_POSTFIELDSIZE, (long)sizeof(g_Data));
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
   test_setopt(curl, CURLOPT_DEBUGDATA, &debug_config);
   test_setopt(curl, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   test_setopt(curl, CURLOPT_VERBOSE, 1L);
@@ -77,14 +77,14 @@ static CURLcode test_lib1522(const char *URL)
     curl_off_t uploadSize;
     curl_easy_getinfo(curl, CURLINFO_SIZE_UPLOAD_T, &uploadSize);
 
-    curl_mprintf("uploadSize = %ld\n", (long)uploadSize);
+    curl_mprintf("uploadSize = %" CURL_FORMAT_CURL_OFF_T "\n", uploadSize);
 
     if((size_t) uploadSize == sizeof(g_Data)) {
       curl_mprintf("!!!!!!!!!! PASS\n");
     }
     else {
-      curl_mprintf("sent %d, libcurl says %d\n",
-                   (int)sizeof(g_Data), (int)uploadSize);
+      curl_mprintf("sent %zu, libcurl says %" CURL_FORMAT_CURL_OFF_T "\n",
+                   sizeof(g_Data), uploadSize);
     }
   }
   else {
index 396ca7901030a1e3ee5e44222fa98fa7767db8e6..9aaec8171a18f032a9e11aa99e9045e92f681725 100644 (file)
@@ -45,17 +45,17 @@ static CURLcode test_lib1538(const char *URL)
   curl_url_strerror((CURLUcode)-INT_MAX);
   /* NOLINTEND(clang-analyzer-optin.core.EnumCastOutOfRange) */
   for(easyret = CURLE_OK; easyret <= CURL_LAST; easyret++) {
-    curl_mprintf("e%d: %s\n", (int)easyret, curl_easy_strerror(easyret));
+    curl_mprintf("e%d: %s\n", easyret, curl_easy_strerror(easyret));
   }
   for(multiret = CURLM_CALL_MULTI_PERFORM; multiret <= CURLM_LAST;
       multiret++) {
-    curl_mprintf("m%d: %s\n", (int)multiret, curl_multi_strerror(multiret));
+    curl_mprintf("m%d: %s\n", multiret, curl_multi_strerror(multiret));
   }
   for(shareret = CURLSHE_OK; shareret <= CURLSHE_LAST; shareret++) {
-    curl_mprintf("s%d: %s\n", (int)shareret, curl_share_strerror(shareret));
+    curl_mprintf("s%d: %s\n", shareret, curl_share_strerror(shareret));
   }
   for(urlret = CURLUE_OK; urlret <= CURLUE_LAST; urlret++) {
-    curl_mprintf("u%d: %s\n", (int)urlret, curl_url_strerror(urlret));
+    curl_mprintf("u%d: %s\n", urlret, curl_url_strerror(urlret));
   }
 
   return res;
index ffd9fbe374b703230d2a0e5ffc71cf6f37fcc831..55195b6ba83476614805dccf2b0387575614cb9c 100644 (file)
@@ -106,8 +106,8 @@ static CURLcode test_lib1540(const char *URL)
   easy_setopt(curls, CURLOPT_XFERINFODATA, &st);
   easy_setopt(curls, CURLOPT_NOPROGRESS, 0L);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
   test_setopt(curls, CURLOPT_DEBUGDATA, &debug_config);
   easy_setopt(curls, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   easy_setopt(curls, CURLOPT_VERBOSE, 1L);
index 51df9edee0890156148d9216a1bc7a1be45d84fc..b8edbcb79710f5205f22f8c2381a0b2f5467dffd 100644 (file)
@@ -46,8 +46,8 @@ static CURLcode test_lib1542(const char *URL)
 
   easy_setopt(easy, CURLOPT_URL, URL);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 0;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = FALSE;
   easy_setopt(easy, CURLOPT_DEBUGDATA, &debug_config);
   easy_setopt(easy, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   easy_setopt(easy, CURLOPT_VERBOSE, 1L);
index e33299034551bf779106091f67a450a62142a39c..5d35adac5c8280409d7b4bc1cf87e64f995f4364 100644 (file)
@@ -71,8 +71,8 @@ static CURLcode test_lib1553(const char *URL)
   easy_setopt(curls, CURLOPT_XFERINFOFUNCTION, t1553_xferinfo);
   easy_setopt(curls, CURLOPT_NOPROGRESS, 1L);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
   test_setopt(curls, CURLOPT_DEBUGDATA, &debug_config);
   easy_setopt(curls, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   easy_setopt(curls, CURLOPT_VERBOSE, 1L);
index 20b86b4e9831f3222acb7b4d260267d50bd3592c..4987fc9f0969a9cbe90a22a85640c024c550c0b8 100644 (file)
@@ -67,7 +67,7 @@ static CURLcode test_lib1556(const char *URL)
     goto test_cleanup;
   }
 
-  curl_mprintf("Max = %ld\n", (long)info.largest);
+  curl_mprintf("Max = %zu\n", info.largest);
 
 test_cleanup:
 
index 2337e7b62d7f995467b6a1fe9b874f4e83571ffd..1aa0f830c75dc199de111d16906ff067fc6451fc 100644 (file)
@@ -59,13 +59,13 @@ static CURLcode test_lib1559(const char *URL)
   if(u) {
     CURLUcode uc = curl_url_set(u, CURLUPART_URL, longurl, 0);
     curl_mprintf("CURLUPART_URL %d bytes URL == %d (%s)\n",
-                 EXCESSIVE, (int)uc, curl_url_strerror(uc));
+                 EXCESSIVE, uc, curl_url_strerror(uc));
     uc = curl_url_set(u, CURLUPART_SCHEME, longurl, CURLU_NON_SUPPORT_SCHEME);
     curl_mprintf("CURLUPART_SCHEME %d bytes scheme == %d (%s)\n",
-                 EXCESSIVE, (int)uc, curl_url_strerror(uc));
+                 EXCESSIVE, uc, curl_url_strerror(uc));
     uc = curl_url_set(u, CURLUPART_USER, longurl, 0);
     curl_mprintf("CURLUPART_USER %d bytes user == %d (%s)\n",
-                 EXCESSIVE, (int)uc, curl_url_strerror(uc));
+                 EXCESSIVE, uc, curl_url_strerror(uc));
     curl_url_cleanup(u);
   }
 
index 2171034d3c76f06cbdadbe8394494af4f7c28199..c0d426c584935e364ccfb33d75d2e13190ff7496 100644 (file)
@@ -73,7 +73,7 @@ static int checkparts(CURLU *u, const char *in, const char *wanted,
       curl_msnprintf(bufp, len, "%s%s", buf[0]?" | ":"", p);
     }
     else
-      curl_msnprintf(bufp, len, "%s[%d]", buf[0]?" | ":"", (int)rc);
+      curl_msnprintf(bufp, len, "%s[%d]", buf[0]?" | ":"", rc);
 
     n = strlen(bufp);
     bufp += n;
@@ -1198,7 +1198,7 @@ static CURLUcode updateurl(CURLU *u, const char *cmd, unsigned int setflags)
         CURLUPart what = part2id(part);
 #if 0
         /* for debugging this */
-        curl_mfprintf(stderr, "%s = \"%s\" [%d]\n", part, value, (int)what);
+        curl_mfprintf(stderr, "%s = \"%s\" [%d]\n", part, value, what);
 #endif
         if(what > CURLUPART_ZONEID)
           curl_mfprintf(stderr, "UNKNOWN part '%s'\n", part);
@@ -1395,7 +1395,7 @@ static int set_url(void)
       if(rc) {
         curl_mfprintf(stderr, "%s:%d Set URL %s returned %d (%s)\n",
                       __FILE__, __LINE__, set_url_list[i].set,
-                      (int)rc, curl_url_strerror(rc));
+                      rc, curl_url_strerror(rc));
         error++;
       }
       else {
@@ -1403,7 +1403,7 @@ static int set_url(void)
         rc = curl_url_get(urlp, CURLUPART_URL, &url, 0);
         if(rc) {
           curl_mfprintf(stderr, "%s:%d Get URL returned %d (%s)\n",
-                        __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                        __FILE__, __LINE__, rc, curl_url_strerror(rc));
           error++;
         }
         else {
@@ -1416,7 +1416,7 @@ static int set_url(void)
     }
     else if(rc != set_url_list[i].ucode) {
       curl_mfprintf(stderr, "Set URL\nin: %s\nreturned %d (expected %d)\n",
-                    set_url_list[i].in, (int)rc, set_url_list[i].ucode);
+                    set_url_list[i].in, rc, set_url_list[i].ucode);
       error++;
     }
     curl_url_cleanup(urlp);
@@ -1455,7 +1455,7 @@ static int setget_parts(bool has_utf8)
         if(uc != setget_parts_list[i].pcode) {
           curl_mfprintf(stderr,
                         "updateurl\nin: %s\nreturned %d (expected %d)\n",
-                        setget_parts_list[i].set, (int)uc,
+                        setget_parts_list[i].set, uc,
                         setget_parts_list[i].pcode);
           error++;
         }
@@ -1470,7 +1470,7 @@ static int setget_parts(bool has_utf8)
       }
       else if(rc != CURLUE_OK) {
         curl_mfprintf(stderr, "Set parts\nin: %s\nreturned %d (expected %d)\n",
-                      setget_parts_list[i].in, (int)rc, 0);
+                      setget_parts_list[i].in, rc, 0);
         error++;
       }
     }
@@ -1503,7 +1503,7 @@ static int set_parts(void)
 
       if(uc != set_parts_list[i].pcode) {
         curl_mfprintf(stderr, "updateurl\nin: %s\nreturned %d (expected %d)\n",
-                      set_parts_list[i].set, (int)uc, set_parts_list[i].pcode);
+                      set_parts_list[i].set, uc, set_parts_list[i].pcode);
         error++;
       }
       if(!uc) {
@@ -1512,7 +1512,7 @@ static int set_parts(void)
 
         if(rc) {
           curl_mfprintf(stderr, "%s:%d Get URL returned %d (%s)\n",
-                        __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                        __FILE__, __LINE__, rc, curl_url_strerror(rc));
           error++;
         }
         else if(checkurl(set_parts_list[i].in, url, set_parts_list[i].out)) {
@@ -1523,7 +1523,7 @@ static int set_parts(void)
     }
     else if(rc != set_parts_list[i].ucode) {
       curl_mfprintf(stderr, "Set parts\nin: %s\nreturned %d (expected %d)\n",
-                    set_parts_list[i].in, (int)rc, set_parts_list[i].ucode);
+                    set_parts_list[i].in, rc, set_parts_list[i].ucode);
       error++;
     }
     curl_url_cleanup(urlp);
@@ -1552,7 +1552,7 @@ static int get_url(bool has_utf8)
 
         if(rc) {
           curl_mfprintf(stderr, "%s:%d returned %d (%s). URL: '%s'\n",
-                        __FILE__, __LINE__, (int)rc, curl_url_strerror(rc),
+                        __FILE__, __LINE__, rc, curl_url_strerror(rc),
                         get_url_list[i].in);
           error++;
         }
@@ -1565,7 +1565,7 @@ static int get_url(bool has_utf8)
       }
       if(rc != get_url_list[i].ucode) {
         curl_mfprintf(stderr, "Get URL\nin: %s\nreturned %d (expected %d)\n",
-                      get_url_list[i].in, (int)rc, get_url_list[i].ucode);
+                      get_url_list[i].in, rc, get_url_list[i].ucode);
         error++;
       }
     }
@@ -1592,7 +1592,7 @@ static int get_parts(bool has_utf8)
                         get_parts_list[i].urlflags);
       if(rc != get_parts_list[i].ucode) {
         curl_mfprintf(stderr, "Get parts\nin: %s\nreturned %d (expected %d)\n",
-                      get_parts_list[i].in, (int)rc, get_parts_list[i].ucode);
+                      get_parts_list[i].in, rc, get_parts_list[i].ucode);
         error++;
       }
       else if(get_parts_list[i].ucode) {
@@ -1649,7 +1649,7 @@ static int append(void)
       ;
     else if(rc != append_list[i].ucode) {
       curl_mfprintf(stderr, "Append\nin: %s\nreturned %d (expected %d)\n",
-                    append_list[i].in, (int)rc, append_list[i].ucode);
+                    append_list[i].in, rc, append_list[i].ucode);
       error++;
     }
     else if(append_list[i].ucode) {
@@ -1660,7 +1660,7 @@ static int append(void)
       rc = curl_url_get(urlp, CURLUPART_URL, &url, 0);
       if(rc) {
         curl_mfprintf(stderr, "%s:%d Get URL returned %d (%s)\n",
-                      __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                      __FILE__, __LINE__, rc, curl_url_strerror(rc));
         error++;
       }
       else {
@@ -1686,7 +1686,7 @@ static int scopeid(void)
                     "https://[fe80::20c:29ff:fe9c:409b%25eth0]/hello.html", 0);
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr, "%s:%d curl_url_set returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
 
@@ -1694,7 +1694,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_get CURLUPART_HOST returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
   else {
@@ -1705,7 +1705,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_set CURLUPART_HOST returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
 
@@ -1713,7 +1713,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_get CURLUPART_URL returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
   else {
@@ -1724,7 +1724,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_set CURLUPART_HOST returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
 
@@ -1732,7 +1732,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_get CURLUPART_URL returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
   else {
@@ -1744,7 +1744,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_set CURLUPART_HOST returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
 
@@ -1752,7 +1752,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_get CURLUPART_URL returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
   else {
@@ -1763,7 +1763,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_get CURLUPART_HOST returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
   else {
@@ -1774,7 +1774,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_get CURLUPART_ZONEID returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
   else {
@@ -1785,7 +1785,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_set CURLUPART_ZONEID returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
 
@@ -1793,7 +1793,7 @@ static int scopeid(void)
   if(rc != CURLUE_OK) {
     curl_mfprintf(stderr,
                   "%s:%d curl_url_get CURLUPART_URL returned %d (%s)\n",
-                  __FILE__, __LINE__, (int)rc, curl_url_strerror(rc));
+                  __FILE__, __LINE__, rc, curl_url_strerror(rc));
     error++;
   }
   else {
@@ -1848,7 +1848,7 @@ static int get_nothing(void)
 
     rc = curl_url_get(u, CURLUPART_ZONEID, &p, 0);
     if(rc != CURLUE_NO_ZONEID)
-      curl_mfprintf(stderr, "unexpected return code %u on line %u\n", (int)rc,
+      curl_mfprintf(stderr, "unexpected return code %u on line %u\n", rc,
                     __LINE__);
 
     curl_url_cleanup(u);
index 4d38da06f2d1552abe52dbff7701b53a0aa46c00..3fecdd84ce4883a1d80b031ba89ebbb2c4ac273b 100644 (file)
@@ -132,7 +132,7 @@ static CURLcode test_lib1565(const char *URL)
       else {
         curl_mfprintf(stderr,
                       "%s:%d Got an unexpected message from curl: %i\n",
-                      __FILE__, __LINE__, (int)message->msg);
+                      __FILE__, __LINE__, message->msg);
         res = TEST_ERR_MAJOR_BAD;
         goto test_cleanup;
       }
index 11088e2edfb9ae82918924d144d2f10cbfcb50bc..2aed892407707409de303df9d3519ea29c382537 100644 (file)
@@ -40,8 +40,8 @@ static CURLcode test_lib1907(const char *URL)
   res = curl_easy_perform(curl);
   if(!res)
     curl_mfprintf(stderr, "failure expected, "
-                  "curl_easy_perform returned %ld: <%s>, <%s>\n",
-                  (long) res, curl_easy_strerror(res), error_buffer);
+                  "curl_easy_perform returned %d: <%s>, <%s>\n",
+                  res, curl_easy_strerror(res), error_buffer);
 
   /* print the used url */
   if(!curl_easy_getinfo(curl, CURLINFO_EFFECTIVE_URL, &url_after))
index 149986f656b7a4d10d6d321579d1d4eeb4a1ca0a..4ffc8085dc12b1a0566253375717ba99a178aed6 100644 (file)
@@ -49,7 +49,7 @@ static CURLcode test_lib1911(const char *URL)
   memset(testbuf, 'A', MAX_INPUT_LENGTH + 1);
   testbuf[MAX_INPUT_LENGTH + 1] = 0;
 
-  curl_mprintf("string length: %d\n", (int)strlen(testbuf));
+  curl_mprintf("string length: %zu\n", strlen(testbuf));
 
   for(o = curl_easy_option_next(NULL);
       o;
index 0f3f135db44d019d60704af6ce61e13af3bf17e6..c4cc39455feb89ce38c00cdd0a8eb9b4db4b4f5c 100644 (file)
@@ -103,8 +103,8 @@ static CURLcode test_lib1915(const char *URL)
 
   global_init(CURL_GLOBAL_ALL);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
 
   easy_init(hnd);
   easy_setopt(hnd, CURLOPT_URL, URL);
index c78564dd1a795bbc1a2bfdb1e4ab7f1ba1a34b3d..50f4010e94eb19649e33e81c2ffcec8c331beb2d 100644 (file)
@@ -59,8 +59,8 @@ static void t1940_showem(CURL *easy, int header_request, unsigned int type)
         size_t index = 0;
         size_t amount = header->amount;
         do {
-          curl_mprintf("- %s == %s (%u/%u)\n", header->name, header->value,
-                       (int)index, (int)amount);
+          curl_mprintf("- %s == %s (%zu/%zu)\n", header->name, header->value,
+                       index, amount);
 
           if(++index == amount)
             break;
index ec280e6da9eb621f24ce12391e7e10e004105b69..6e693bda625ddbaa74af8cf82a3980e3556248e4 100644 (file)
@@ -32,8 +32,8 @@ static void t1945_showem(CURL *easy, unsigned int type)
 
   /* !checksrc! disable EQUALSNULL 1 */
   while((header = curl_easy_nextheader(easy, type, 0, prev)) != NULL) {
-    curl_mprintf(" %s == %s (%u/%u)\n", header->name, header->value,
-                 (int)header->index, (int)header->amount);
+    curl_mprintf(" %s == %s (%zu/%zu)\n", header->name, header->value,
+                 header->index, header->amount);
     prev = header;
   }
 }
index 5bca7d87285ec7e6c1db72332821bc72e3023da6..88f4ad3b741b6031c1d09a77b67853d150461d21 100644 (file)
@@ -49,7 +49,7 @@ static size_t callback(char *ptr, size_t size, size_t nmemb, void *data)
   if(CURLE_OK != code) {
     curl_mfprintf(stderr, "%s:%d curl_easy_getinfo() failed, "
                   "with code %d (%s)\n",
-                  __FILE__, __LINE__, (int)code, curl_easy_strerror(code));
+                  __FILE__, __LINE__, code, curl_easy_strerror(code));
     ntlmcb_res = TEST_ERR_MAJOR_BAD;
     return failure;
   }
index 3dd198115b92dfec6c8e9a9fdedc98ed8e98ccba..d99f7dab068ab5dacdb47344b3da61ad91acd2cb 100644 (file)
@@ -50,8 +50,8 @@ static size_t t2301_write_cb(char *b, size_t size, size_t nitems, void *p)
     0x8a, 0x0
   };
   size_t incoming = nitems;
-  curl_mfprintf(stderr, "Called CURLOPT_WRITEFUNCTION with %d bytes: ",
-                (int)nitems);
+  curl_mfprintf(stderr, "Called CURLOPT_WRITEFUNCTION with %zu bytes: ",
+                nitems);
   for(i = 0; i < nitems; i++)
     curl_mfprintf(stderr, "%02x ", (unsigned char)buffer[i]);
   curl_mfprintf(stderr, "\n");
index fcc6078a2edddf9b64c463690bb630cad324c9e2..ef3dc226a09b56b654c9ddad7b1a584c033202ce 100644 (file)
@@ -63,8 +63,8 @@ static CURLcode test_lib2402(const char *URL)
     easy_init(curl[i]);
     /* specify target */
     curl_msnprintf(target_url, sizeof(target_url),
-                   "https://localhost:%s/path/2402%04i",
-                   port, (int)i + 1);
+                   "https://localhost:%s/path/2402%04zu",
+                   port, i + 1);
     target_url[sizeof(target_url) - 1] = '\0';
     easy_setopt(curl[i], CURLOPT_URL, target_url);
     /* go http2 */
index 3a3597010861a100116ab9fa23879d65c198da57..8551957e0b4f5113f4f50bdafe3bed53c4bcdd6d 100644 (file)
@@ -63,8 +63,8 @@ static CURLcode test_lib2404(const char *URL)
     easy_init(curl[i]);
     /* specify target */
     curl_msnprintf(target_url, sizeof(target_url),
-                   "https://localhost:%s/path/2404%04i",
-                   port, (int)i + 1);
+                   "https://localhost:%s/path/2404%04zu",
+                   port, i + 1);
     target_url[sizeof(target_url) - 1] = '\0';
     easy_setopt(curl[i], CURLOPT_URL, target_url);
     /* go http2 */
index 2ae7129cf6912c27102aa0a1e3a4bb9538b21d3d..ba7da99a07065aa4d32a3bae8e35e0dd199a79ce 100644 (file)
@@ -64,8 +64,8 @@ static CURLcode test_lib2502(const char *URL)
     easy_init(curl[i]);
     /* specify target */
     curl_msnprintf(target_url, sizeof(target_url),
-                   "https://localhost:%s/path/2502%04i",
-                   port, (int)i + 1);
+                   "https://localhost:%s/path/2502%04zu",
+                   port, i + 1);
     target_url[sizeof(target_url) - 1] = '\0';
     easy_setopt(curl[i], CURLOPT_URL, target_url);
     /* go http2 */
@@ -75,8 +75,8 @@ static CURLcode test_lib2502(const char *URL)
     /* wait for first connection established to see if we can share it */
     easy_setopt(curl[i], CURLOPT_PIPEWAIT, 1L);
     /* go verbose */
-    debug_config.nohex = 1;
-    debug_config.tracetime = 0;
+    debug_config.nohex = TRUE;
+    debug_config.tracetime = FALSE;
     test_setopt(curl[i], CURLOPT_DEBUGDATA, &debug_config);
     easy_setopt(curl[i], CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
     easy_setopt(curl[i], CURLOPT_VERBOSE, 1L);
index c3b2cc42f4247dcd1a2d137fa394fe9c817beffc..c0d2fdb642088cb433960d5810fc17388759432f 100644 (file)
@@ -222,8 +222,8 @@ static CURLcode test_lib2700(const char *URL)
 
   easy_setopt(curl, CURLOPT_URL, URL);
   easy_setopt(curl, CURLOPT_USERAGENT, "client/test2700");
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
   easy_setopt(curl, CURLOPT_DEBUGDATA, &debug_config);
   easy_setopt(curl, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   easy_setopt(curl, CURLOPT_VERBOSE, 1L);
index 13254098d23257db22fe88010ae03f5ef1573290..2b35a258417a7ce0262c14d1009bb2ff62aebc7e 100644 (file)
@@ -89,7 +89,7 @@ cleanup:
     if(results[i] != CURLE_OK) {
       curl_mfprintf(stderr, "%s:%d thread[%u]: curl_global_init() failed,"
                     "with code %d (%s)\n", __FILE__, __LINE__,
-                    i, (int) results[i], curl_easy_strerror(results[i]));
+                    i, results[i], curl_easy_strerror(results[i]));
       test_failure = TEST_ERR_MAJOR_BAD;
     }
   }
@@ -147,7 +147,7 @@ cleanup:
     if(results[i] != CURLE_OK) {
       curl_mfprintf(stderr, "%s:%d thread[%u]: curl_global_init() failed,"
                     "with code %d (%s)\n", __FILE__, __LINE__,
-                    i, (int) results[i], curl_easy_strerror(results[i]));
+                    i, results[i], curl_easy_strerror(results[i]));
       test_failure = TEST_ERR_MAJOR_BAD;
     }
   }
index f8ac085befbf5e7fdb8f031f4aec9b313ca0d5fd..692541329e66fde969176d406366955bf4bc8d71 100644 (file)
@@ -107,8 +107,8 @@ static CURLcode test_lib3033(const char *URL)
   multi_init(multi);
   easy_init(curl);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
 
   res = t3033_req_test(multi, curl, URL, 0);
   if(res != CURLE_OK)
index 5c231c2f82ef9c8011eda17a949fc10f15120294..29dfdae00c3719c0bc5576bb7b7c6328f979552b 100644 (file)
@@ -47,7 +47,7 @@ static CURLcode test_lib3105(const char *URL)
   mc += curl_multi_remove_handle(multi, curls);
 
   if(mc) {
-    curl_mfprintf(stderr, "%d was unexpected\n", (int)mc);
+    curl_mfprintf(stderr, "%d was unexpected\n", mc);
     i = CURLE_FAILED_INIT;
   }
 
index bc119d5f4146191123b66df86f77ef2249a81086..0586c1407f9c0190e364d28da66c17ae65bdb133 100644 (file)
@@ -73,8 +73,8 @@ static CURLcode test_lib500(const char *URL)
   test_setopt(curl, CURLOPT_URL, URL);
   test_setopt(curl, CURLOPT_HEADER, 1L);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
   test_setopt(curl, CURLOPT_DEBUGDATA, &debug_config);
   test_setopt(curl, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   test_setopt(curl, CURLOPT_VERBOSE, 1L);
index ba82c9a42ddcbbd3f507ace608fad910687e392b..79d9dbca551c49b9489bfa6ada02801bd7500f2c 100644 (file)
@@ -26,7 +26,6 @@
 #include "testutil.h"
 #include "memdebug.h"
 
-#define JAR libtest_arg2
 #define THREADS 2
 
 /* struct containing data of a thread */
@@ -152,7 +151,7 @@ static void *t506_test_fire(void *ptr)
   if(code) {
     int i = 0;
     curl_mfprintf(stderr, "perform url '%s' repeat %d failed, curlcode %d\n",
-                  tdata->url, i, (int)code);
+                  tdata->url, i, code);
   }
 
   curl_mprintf("CLEANUP\n");
@@ -178,6 +177,8 @@ static CURLcode test_lib506(const char *URL)
   int i;
   struct t506_userdata user;
 
+  const char *jar = libtest_arg2;
+
   user.text = "Pigs in space";
   user.counter = 0;
 
@@ -284,7 +285,7 @@ static CURLcode test_lib506(const char *URL)
   curl_mprintf("CURLOPT_SHARE\n");
   test_setopt(curl, CURLOPT_SHARE,      share);
   curl_mprintf("CURLOPT_COOKIEJAR\n");
-  test_setopt(curl, CURLOPT_COOKIEJAR,  JAR);
+  test_setopt(curl, CURLOPT_COOKIEJAR,  jar);
   curl_mprintf("CURLOPT_COOKIELIST FLUSH\n");
   test_setopt(curl, CURLOPT_COOKIELIST, "FLUSH");
 
@@ -313,7 +314,7 @@ static CURLcode test_lib506(const char *URL)
   curl_mprintf("CURLOPT_COOKIELIST ALL\n");
   test_setopt(curl, CURLOPT_COOKIELIST, "ALL");
   curl_mprintf("CURLOPT_COOKIEJAR\n");
-  test_setopt(curl, CURLOPT_COOKIEFILE, JAR);
+  test_setopt(curl, CURLOPT_COOKIEFILE, jar);
   curl_mprintf("CURLOPT_COOKIELIST RELOAD\n");
   test_setopt(curl, CURLOPT_COOKIELIST, "RELOAD");
 
@@ -327,7 +328,7 @@ static CURLcode test_lib506(const char *URL)
   }
   curl_mprintf("loaded cookies:\n");
   if(!cookies) {
-    curl_mfprintf(stderr, "  reloading cookies from '%s' failed\n", JAR);
+    curl_mfprintf(stderr, "  reloading cookies from '%s' failed\n", jar);
     res = TEST_ERR_MAJOR_BAD;
     goto test_cleanup;
   }
@@ -363,8 +364,7 @@ test_cleanup:
   curl_mprintf("SHARE_CLEANUP\n");
   scode = curl_share_cleanup(share);
   if(scode != CURLSHE_OK)
-    curl_mfprintf(stderr, "curl_share_cleanup failed, code errno %d\n",
-                  (int)scode);
+    curl_mfprintf(stderr, "curl_share_cleanup failed, code errno %d\n", scode);
 
   curl_mprintf("GLOBAL_CLEANUP\n");
   curl_global_cleanup();
index 8cf0687441066cb24251d972d68f4deccff5c708..963181713b27b1130ea3546173e6838e067bdca4 100644 (file)
@@ -96,7 +96,7 @@ static CURLcode test_lib526(const char *URL)
         curl[current] = NULL;
       }
       if(++current < CURL_ARRAYSIZE(curl)) {
-        curl_mfprintf(stderr, "Advancing to URL %d\n", (int)current);
+        curl_mfprintf(stderr, "Advancing to URL %zu\n", current);
         if(testnum == 532) {
           /* first remove the only handle we use */
           curl_multi_remove_handle(m, curl[0]);
index 4ac5b1607249789faf51526e6671cf83cb625bd3..80268678209c3524072c5618027b15a6b32fe18e 100644 (file)
@@ -206,7 +206,7 @@ static int t530_checkForCompletion(CURLM *curl, int *success)
     }
     else {
       curl_mfprintf(stderr, "%s got an unexpected message from curl: %i\n",
-                    t530_tag(), (int)message->msg);
+                    t530_tag(), message->msg);
       result = 1;
       *success = 0;
     }
index 6c8b8ee1d858b4f5c9168788249146411784f38b..c0107ed9cbdce909f4c9c7c791884dd5b6ddea0b 100644 (file)
@@ -25,9 +25,6 @@
 
 #include "memdebug.h"
 
-#define WITH_PROXY     "http://usingproxy.com/"
-#define WITHOUT_PROXY  libtest_arg2
-
 static void proxystat(CURL *curl)
 {
   long wasproxy;
@@ -43,6 +40,9 @@ static CURLcode test_lib536(const char *URL)
   CURL *curl;
   struct curl_slist *host = NULL;
 
+  static const char *url_with_proxy = "http://usingproxy.com/";
+  const char *url_without_proxy = libtest_arg2;
+
   if(curl_global_init(CURL_GLOBAL_ALL) != CURLE_OK) {
     curl_mfprintf(stderr, "curl_global_init() failed\n");
     return TEST_ERR_MAJOR_BAD;
@@ -61,14 +61,14 @@ static CURLcode test_lib536(const char *URL)
 
   test_setopt(curl, CURLOPT_RESOLVE, host);
   test_setopt(curl, CURLOPT_PROXY, URL);
-  test_setopt(curl, CURLOPT_URL, WITH_PROXY);
+  test_setopt(curl, CURLOPT_URL, url_with_proxy);
   test_setopt(curl, CURLOPT_NOPROXY, "goingdirect.com");
   test_setopt(curl, CURLOPT_VERBOSE, 1L);
 
   res = curl_easy_perform(curl);
   if(!res) {
     proxystat(curl);
-    test_setopt(curl, CURLOPT_URL, WITHOUT_PROXY);
+    test_setopt(curl, CURLOPT_URL, url_without_proxy);
     res = curl_easy_perform(curl);
     if(!res)
       proxystat(curl);
index 22611189e2a6a2d5ed5178612a69faa1ea09ab34..5dc6628079df06d00253ed9f47cc0072538b5959 100644 (file)
 
 #include "memdebug.h"
 
-#define PROXY libtest_arg2
-#define PROXYUSERPWD libtest_arg3
-#define HOST test_argv[4]
-
 static CURL *testeh[2];
 
 static CURLcode init(int num, CURLM *cm, const char *url, const char *userpwd,
@@ -45,6 +41,8 @@ static CURLcode init(int num, CURLM *cm, const char *url, const char *userpwd,
 {
   CURLcode res = CURLE_OK;
 
+  const char *proxy = libtest_arg2;
+
   res_easy_init(testeh[num]);
   if(res)
     goto init_failed;
@@ -53,7 +51,7 @@ static CURLcode init(int num, CURLM *cm, const char *url, const char *userpwd,
   if(res)
     goto init_failed;
 
-  res_easy_setopt(testeh[num], CURLOPT_PROXY, PROXY);
+  res_easy_setopt(testeh[num], CURLOPT_PROXY, proxy);
   if(res)
     goto init_failed;
 
@@ -173,7 +171,7 @@ static CURLcode loop(int num, CURLM *cm, const char *url, const char *userpwd,
         }
       }
       else
-        curl_mfprintf(stderr, "E: CURLMsg (%d)\n", (int)msg->msg);
+        curl_mfprintf(stderr, "E: CURLMsg (%d)\n", msg->msg);
     }
 
     res_test_timedout();
@@ -192,6 +190,9 @@ static CURLcode test_lib540(const char *URL)
   CURLcode res = CURLE_OK;
   size_t i;
 
+  const char *proxyuserpws = libtest_arg3;
+  const char *host;
+
   for(i = 0; i < CURL_ARRAYSIZE(testeh); i++)
     testeh[i] = NULL;
 
@@ -200,7 +201,8 @@ static CURLcode test_lib540(const char *URL)
   if(test_argc < 4)
     return TEST_ERR_MAJOR_BAD;
 
-  curl_msnprintf(buffer, sizeof(buffer), "Host: %s", HOST);
+  host = test_argv[4];
+  curl_msnprintf(buffer, sizeof(buffer), "Host: %s", host);
 
   /* now add a custom Host: header */
   headers = curl_slist_append(headers, buffer);
@@ -222,13 +224,13 @@ static CURLcode test_lib540(const char *URL)
     return res;
   }
 
-  res = loop(0, cm, URL, PROXYUSERPWD, headers);
+  res = loop(0, cm, URL, proxyuserpws, headers);
   if(res)
     goto test_cleanup;
 
   curl_mfprintf(stderr, "lib540: now we do the request again\n");
 
-  res = loop(1, cm, URL, PROXYUSERPWD, headers);
+  res = loop(1, cm, URL, proxyuserpws, headers);
 
 test_cleanup:
 
index f5dcb5383c9f5d1b78c03c76f6dc364a93a85fc7..c49a8e54b8f496495aa17a0591909a71142c325a 100644 (file)
@@ -56,7 +56,7 @@ static CURLcode test_lib544(const char *URL)
   test_setopt(curl, CURLOPT_URL, URL);
 
   if(testnum == 545)
-    test_setopt(curl, CURLOPT_POSTFIELDSIZE, (long) sizeof(teststring));
+    test_setopt(curl, CURLOPT_POSTFIELDSIZE, (long)sizeof(teststring));
 
   test_setopt(curl, CURLOPT_COPYPOSTFIELDS, teststring);
 
index 8892dab028b6ac2ac63690d2e671ba4f867e4737..e3597d6d3e967d2b28a3b6a8639010c1b2873b0b 100644 (file)
@@ -30,7 +30,8 @@
 
 #include "memdebug.h"
 
-#define UPLOADTHIS "this is the blurb we want to upload\n"
+static const char t547_uploadthis[] = "this is the blurb we want to upload\n";
+#define T547_DATALEN (sizeof(t547_uploadthis)-1)
 
 static size_t t547_read_cb(char *ptr, size_t size, size_t nmemb, void *clientp)
 {
@@ -43,10 +44,10 @@ static size_t t547_read_cb(char *ptr, size_t size, size_t nmemb, void *clientp)
   }
   (*counter)++; /* bump */
 
-  if(size * nmemb >= strlen(UPLOADTHIS)) {
+  if(size * nmemb >= T547_DATALEN) {
     curl_mfprintf(stderr, "READ!\n");
-    strcpy(ptr, UPLOADTHIS);
-    return strlen(UPLOADTHIS);
+    strcpy(ptr, t547_uploadthis);
+    return T547_DATALEN;
   }
   curl_mfprintf(stderr, "READ NOT FINE!\n");
   return 0;
@@ -86,7 +87,7 @@ static CURLcode test_lib547(const char *URL)
   test_setopt(curl, CURLOPT_HEADER, 1L);
   if(testnum == 548) {
     /* set the data to POST with a mere pointer to a null-terminated string */
-    test_setopt(curl, CURLOPT_POSTFIELDS, UPLOADTHIS);
+    test_setopt(curl, CURLOPT_POSTFIELDS, t547_uploadthis);
   }
   else {
     /* 547 style, which means reading the POST data from a callback */
@@ -97,7 +98,7 @@ static CURLcode test_lib547(const char *URL)
     test_setopt(curl, CURLOPT_READDATA, &counter);
     /* We CANNOT do the POST fine without setting the size (or choose
        chunked)! */
-    test_setopt(curl, CURLOPT_POSTFIELDSIZE, (long)strlen(UPLOADTHIS));
+    test_setopt(curl, CURLOPT_POSTFIELDSIZE, (long)T547_DATALEN);
   }
   test_setopt(curl, CURLOPT_POST, 1L);
   test_setopt(curl, CURLOPT_PROXY, libtest_arg2);
index 5dec1949c389ccbc6b98d689475198c40e3b9307..eb42f35b544550493e6e2568076f7bfe2b7a8768 100644 (file)
@@ -74,8 +74,8 @@ static CURLcode test_lib552(const char *URL)
   size_t i;
   static const char fill[] = "test data";
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 0;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = FALSE;
 
   global_init(CURL_GLOBAL_ALL);
   easy_init(curl);
@@ -93,7 +93,7 @@ static CURLcode test_lib552(const char *URL)
   test_setopt(curl, CURLOPT_POST, 1L);
 
   /* Setup read callback */
-  test_setopt(curl, CURLOPT_POSTFIELDSIZE, (long) sizeof(databuf));
+  test_setopt(curl, CURLOPT_POSTFIELDSIZE, (long)sizeof(databuf));
   test_setopt(curl, CURLOPT_READFUNCTION, t552_read_cb);
 
   /* Write callback */
index f705fbbd3ead0f4443b2ac42b82ad26ac67a5f20..23cb5a7046adce1058bd84883e2fb28bb87d25bb 100644 (file)
@@ -95,7 +95,7 @@ static CURLcode t554_test_once(const char *URL, bool oldstyle)
   }
 
   if(formrc)
-    curl_mprintf("curl_formadd(1) = %d\n", (int)formrc);
+    curl_mprintf("curl_formadd(1) = %d\n", formrc);
 
   /* Now add the same data with another name and make it not look like
      a file upload but still using the callback */
@@ -112,7 +112,7 @@ static CURLcode t554_test_once(const char *URL, bool oldstyle)
                         CURLFORM_END);
 
   if(formrc)
-    curl_mprintf("curl_formadd(2) = %d\n", (int)formrc);
+    curl_mprintf("curl_formadd(2) = %d\n", formrc);
 
   /* Fill in the filename field */
   formrc = curl_formadd(&formpost,
@@ -121,7 +121,7 @@ static CURLcode t554_test_once(const char *URL, bool oldstyle)
                         CURLFORM_COPYCONTENTS, "postit2.c",
                         CURLFORM_END);
   if(formrc)
-    curl_mprintf("curl_formadd(3) = %d\n", (int)formrc);
+    curl_mprintf("curl_formadd(3) = %d\n", formrc);
 
   /* Fill in a submit field too */
   formrc = curl_formadd(&formpost,
@@ -132,17 +132,17 @@ static CURLcode t554_test_once(const char *URL, bool oldstyle)
                         CURLFORM_END);
 
   if(formrc)
-    curl_mprintf("curl_formadd(4) = %d\n", (int)formrc);
+    curl_mprintf("curl_formadd(4) = %d\n", formrc);
 
   formrc = curl_formadd(&formpost, &lastptr,
                         CURLFORM_COPYNAME, "somename",
                         CURLFORM_BUFFER, "somefile.txt",
                         CURLFORM_BUFFERPTR, "blah blah",
-                        CURLFORM_BUFFERLENGTH, (long)9,
+                        CURLFORM_BUFFERLENGTH, 9L,
                         CURLFORM_END);
 
   if(formrc)
-    curl_mprintf("curl_formadd(5) = %d\n", (int)formrc);
+    curl_mprintf("curl_formadd(5) = %d\n", formrc);
 
   curl = curl_easy_init();
   if(!curl) {
index 97a5d874b3d9ecaeb96ec2dd2400927ad1f2cd93..30438bafa9d4b9f336ad97b4ac6bc5efa7adf0a0 100644 (file)
@@ -34,7 +34,8 @@
 
 #include "memdebug.h"
 
-static const char uploadthis[] = "this is the blurb we want to upload\n";
+static const char t555_uploadthis[] = "this is the blurb we want to upload\n";
+#define T555_DATALEN (sizeof(t555_uploadthis)-1)
 
 static size_t t555_read_cb(char *ptr, size_t size, size_t nmemb, void *clientp)
 {
@@ -47,10 +48,10 @@ static size_t t555_read_cb(char *ptr, size_t size, size_t nmemb, void *clientp)
   }
   (*counter)++; /* bump */
 
-  if(size * nmemb >= strlen(uploadthis)) {
+  if(size * nmemb >= T555_DATALEN) {
     curl_mfprintf(stderr, "READ!\n");
-    strcpy(ptr, uploadthis);
-    return strlen(uploadthis);
+    strcpy(ptr, t555_uploadthis);
+    return T555_DATALEN;
   }
   curl_mfprintf(stderr, "READ NOT FINE!\n");
   return 0;
@@ -93,7 +94,7 @@ static CURLcode test_lib555(const char *URL)
   easy_setopt(curl, CURLOPT_READDATA, &counter);
   /* We CANNOT do the POST fine without setting the size (or choose
      chunked)! */
-  easy_setopt(curl, CURLOPT_POSTFIELDSIZE, (long)strlen(uploadthis));
+  easy_setopt(curl, CURLOPT_POSTFIELDSIZE, (long)T555_DATALEN);
 
   easy_setopt(curl, CURLOPT_POST, 1L);
   easy_setopt(curl, CURLOPT_PROXY, libtest_arg2);
index 8c12e960f76db4a03313826eefae9eb8a58a099b..61b051a43c5f4d7c7cc9eca19538aaca8dc61a3b 100644 (file)
 #endif
 
 #define BUFSZ    256
-#define USHORT_TESTS_ARRSZ 1 + 100
-#define SSHORT_TESTS_ARRSZ 1 + 100
-#define UINT_TESTS_ARRSZ   1 + 100
-#define SINT_TESTS_ARRSZ   1 + 100
-#define ULONG_TESTS_ARRSZ  1 + 100
-#define SLONG_TESTS_ARRSZ  1 + 100
-#define COFFT_TESTS_ARRSZ  1 + 100
 
 
 struct unsshort_st {
@@ -103,13 +96,13 @@ struct curloff_st {
 };
 
 
-static struct unsshort_st us_test[USHORT_TESTS_ARRSZ];
-static struct sigshort_st ss_test[SSHORT_TESTS_ARRSZ];
-static struct unsint_st   ui_test[UINT_TESTS_ARRSZ];
-static struct sigint_st   si_test[SINT_TESTS_ARRSZ];
-static struct unslong_st  ul_test[ULONG_TESTS_ARRSZ];
-static struct siglong_st  sl_test[SLONG_TESTS_ARRSZ];
-static struct curloff_st  co_test[COFFT_TESTS_ARRSZ];
+static struct unsshort_st us_test[1 + 100];
+static struct sigshort_st ss_test[1 + 100];
+static struct unsint_st   ui_test[1 + 100];
+static struct sigint_st   si_test[1 + 100];
+static struct unslong_st  ul_test[1 + 100];
+static struct siglong_st  sl_test[1 + 100];
+static struct curloff_st  co_test[1 + 100];
 
 
 static int test_unsigned_short_formatting(void)
index 0350fb36cd362d6f264f10024a987260b38dc13a..a24a69bc403bcc3d95f482cc325c2593cf54700e 100644 (file)
@@ -62,8 +62,7 @@ static size_t rtp_write(char *ptr, size_t size, size_t nmemb, void *stream)
   curl_mprintf("RTP: message size %d, channel %d\n", message_size, channel);
   if(message_size != coded_size) {
     curl_mprintf("RTP embedded size (%d) does not match "
-                 "the write size (%d).\n",
-                 coded_size, message_size);
+                 "the write size (%d).\n", coded_size, message_size);
     return failure;
   }
 
index 5e291fa19833e5bc64433295d912dd85d5779484..572bfa0cd12c145c17c0c96bcbd5106dcaa4d4e6 100644 (file)
@@ -53,8 +53,8 @@ static CURLcode test_lib573(const char *URL)
   easy_setopt(c, CURLOPT_HEADER, 1L);
   easy_setopt(c, CURLOPT_URL, URL);
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
   easy_setopt(c, CURLOPT_DEBUGDATA, &debug_config);
   easy_setopt(c, CURLOPT_DEBUGFUNCTION, libtest_debug_cb);
   easy_setopt(c, CURLOPT_VERBOSE, 1L);
index 23ec30e322abb9f04302191297eadabc617ab999..e82be6c0d6630ec208622bf0bf15e83ebdb7c67c 100644 (file)
@@ -54,7 +54,7 @@ static CURLcode test_lib574(const char *URL)
   test_setopt(curl, CURLOPT_URL, URL);
   test_setopt(curl, CURLOPT_WILDCARDMATCH, 1L);
   test_setopt(curl, CURLOPT_FNMATCH_FUNCTION, new_fnmatch);
-  test_setopt(curl, CURLOPT_TIMEOUT_MS, (long) TEST_HANG_TIMEOUT);
+  test_setopt(curl, CURLOPT_TIMEOUT_MS, (long)TEST_HANG_TIMEOUT);
 
   res = curl_easy_perform(curl);
   if(res) {
index d0fedd817d55157cbdf2e44d38b098fa1f6854d5..a054dba3fa914026ccd043575b7043a5a6b05820 100644 (file)
@@ -46,7 +46,7 @@ static long chunk_bgn(const void *f, void *ptr, int remains)
       curl_mprintf(" (parsed => %o)", finfo->perm);
     curl_mprintf("\n");
   }
-  curl_mprintf("Size:         %ldB\n", (long)finfo->size);
+  curl_mprintf("Size:         %" CURL_FORMAT_CURL_OFF_T "B\n", finfo->size);
   if(finfo->strings.user)
     curl_mprintf("User:         %s\n", finfo->strings.user);
   if(finfo->strings.group)
index f511c0c3a212b2fc8cf1a72f3e6452a67ed6c0e4..7bff57c2ccb6c42fa8459de372c99fb9e981c4fe 100644 (file)
@@ -44,7 +44,7 @@ static void t582_removeFd(struct t582_Sockets *sockets, curl_socket_t fd,
   int i;
 
   if(mention)
-    curl_mfprintf(stderr, "Remove socket fd %d\n", (int) fd);
+    curl_mfprintf(stderr, "Remove socket fd %d\n", (int)fd);
 
   for(i = 0; i < sockets->count; ++i) {
     if(sockets->sockets[i] == fd) {
@@ -66,7 +66,7 @@ static void t582_addFd(struct t582_Sockets *sockets, curl_socket_t fd,
    * To ensure we only have each file descriptor once, we remove it then add
    * it again.
    */
-  curl_mfprintf(stderr, "Add socket fd %d for %s\n", (int) fd, what);
+  curl_mfprintf(stderr, "Add socket fd %d for %s\n", (int)fd, what);
   t582_removeFd(sockets, fd, 0);
   /*
    * Allocate array storage when required.
@@ -153,7 +153,7 @@ static int t582_checkForCompletion(CURLM *curl, int *success)
     }
     else {
       curl_mfprintf(stderr, "Got an unexpected message from curl: %i\n",
-                    (int)message->msg);
+                    message->msg);
       result = 1;
       *success = 0;
     }
@@ -265,7 +265,8 @@ static CURLcode test_lib582(const char *URL)
     fclose(hd_src);
     return TEST_ERR_FSTAT;
   }
-  curl_mfprintf(stderr, "Set to upload %d bytes\n", (int)file_info.st_size);
+  curl_mfprintf(stderr, "Set to upload %" CURL_FORMAT_CURL_OFF_T " bytes\n",
+                (curl_off_t)file_info.st_size);
 
   res_global_init(CURL_GLOBAL_ALL);
   if(res != CURLE_OK) {
index 10cb77689e3b1f35cf1466201562e77da151d4b0..9950e8b402b73377e7ea6fce186cbb3f95a9ce48 100644 (file)
@@ -70,8 +70,8 @@ static CURLcode test_lib583(const char *URL)
   curl_mfprintf(stderr, "curl_multi_remove_handle()...\n");
   mres = curl_multi_remove_handle(multiHandle, curl);
   if(mres) {
-    curl_mfprintf(stderr, "curl_multi_remove_handle() failed, "
-                  "with code %d\n", (int)mres);
+    curl_mfprintf(stderr, "curl_multi_remove_handle() failed, with code %d\n",
+                  mres);
     res = TEST_ERR_MULTI;
   }
   else
index ccde713ce33f32ecd85741d1a13270791e0ab563..3fb2bf1985071adc4c05ca25e2f447c85de13829 100644 (file)
@@ -120,7 +120,7 @@ static void *t586_test_fire(void *ptr)
   if(code != CURLE_OK) {
     int i = 0;
     curl_mfprintf(stderr, "perform url '%s' repeat %d failed, curlcode %d\n",
-                  tdata->url, i, (int)code);
+                  tdata->url, i, code);
   }
 
   curl_mprintf("CLEANUP\n");
@@ -236,7 +236,7 @@ test_cleanup:
   scode = curl_share_cleanup(share);
   if(scode != CURLSHE_OK)
     curl_mfprintf(stderr, "curl_share_cleanup failed, code errno %d\n",
-                  (int)scode);
+                  scode);
 
   curl_mprintf("GLOBAL_CLEANUP\n");
   curl_global_cleanup();
index e758aabac058288e1f073a30444ad6da47a0ae93..069e94f8e5cec4c0104fee55e4df2ee51b581bd2 100644 (file)
@@ -82,7 +82,7 @@ static CURLcode test_lib650(const char *URL)
                         CURLFORM_CONTENTHEADER, headers,
                         CURLFORM_END);
   if(formrc) {
-    curl_mprintf("curl_formadd(1) = %d\n", (int) formrc);
+    curl_mprintf("curl_formadd(1) = %d\n", formrc);
     goto test_cleanup;
   }
 
@@ -104,7 +104,7 @@ static CURLcode test_lib650(const char *URL)
                         CURLFORM_END);
 
   if(formrc) {
-    curl_mprintf("curl_formadd(2) = %d\n", (int) formrc);
+    curl_mprintf("curl_formadd(2) = %d\n", formrc);
     goto test_cleanup;
   }
 
@@ -124,7 +124,7 @@ static CURLcode test_lib650(const char *URL)
                         CURLFORM_END);
 
   if(formrc) {
-    curl_mprintf("curl_formadd(3) = %d\n", (int) formrc);
+    curl_mprintf("curl_formadd(3) = %d\n", formrc);
     goto test_cleanup;
   }
 
@@ -135,7 +135,7 @@ static CURLcode test_lib650(const char *URL)
                         CURLFORM_FILECONTENT, libtest_arg2,
                         CURLFORM_END);
   if(formrc) {
-    curl_mprintf("curl_formadd(4) = %d\n", (int) formrc);
+    curl_mprintf("curl_formadd(4) = %d\n", formrc);
     goto test_cleanup;
   }
 
@@ -146,7 +146,7 @@ static CURLcode test_lib650(const char *URL)
   curl_formget(formpost, (void *) &formlength, count_chars);
 
   /* Include length in data for external check. */
-  curl_msnprintf(flbuf, sizeof(flbuf), "%lu", (unsigned long) formlength);
+  curl_msnprintf(flbuf, sizeof(flbuf), "%zu", formlength);
 
   formrc = curl_formadd(&formpost,
                         &lastptr,
@@ -155,7 +155,7 @@ static CURLcode test_lib650(const char *URL)
                         CURLFORM_END);
 
   if(formrc) {
-    curl_mprintf("curl_formadd(5) = %d\n", (int) formrc);
+    curl_mprintf("curl_formadd(5) = %d\n", formrc);
     goto test_cleanup;
   }
 
@@ -167,7 +167,7 @@ static CURLcode test_lib650(const char *URL)
                         CURLFORM_END);
 
   if(formrc) {
-    curl_mprintf("curl_formadd(6) = %d\n", (int) formrc);
+    curl_mprintf("curl_formadd(6) = %d\n", formrc);
     goto test_cleanup;
   }
 
index 904eafc6268324d32d9d533ece512610f6b94863..05e9e381da9cd84b4d33d1514a79d4e47dd94edd 100644 (file)
@@ -55,7 +55,7 @@ static CURLcode test_lib651(const char *URL)
                         CURLFORM_COPYCONTENTS, testbuf,
                         CURLFORM_END);
   if(formrc)
-    curl_mprintf("curl_formadd(1) = %d\n", (int) formrc);
+    curl_mprintf("curl_formadd(1) = %d\n", formrc);
 
 
   curl = curl_easy_init();
index 6742b8e4d9f4ee27a3a65206a1a52eed02364165..9dcdd7fd45e8a0e7e62400ad8666dd5f6916dbc9 100644 (file)
@@ -125,8 +125,8 @@ static CURLcode test_lib670(const char *URL)
     res = curl_mime_name(part, testname);
     if(res != CURLE_OK) {
       curl_mfprintf(stderr,
-              "Something went wrong when building the mime structure: %d\n",
-              res);
+                 "Something went wrong when building the mime structure: %d\n",
+                 res);
       goto test_cleanup;
     }
 
@@ -147,7 +147,7 @@ static CURLcode test_lib670(const char *URL)
                           CURLFORM_CONTENTLEN, (curl_off_t) 2,
                           CURLFORM_END);
     if(formrc) {
-      curl_mfprintf(stderr, "curl_formadd() = %d\n", (int) formrc);
+      curl_mfprintf(stderr, "curl_formadd() = %d\n", formrc);
       goto test_cleanup;
     }
 
index 8776c30d812c4d0fa5f291bd23e57fb5076e6f6c..526e1706c376a0509d0e62cbd5d55f4764b10b92 100644 (file)
@@ -87,7 +87,7 @@ static CURLcode test_lib677(const char *URL)
         }
         else if(ec) {
           curl_mfprintf(stderr, "curl_easy_send() failed, with code %d (%s)\n",
-                        (int)ec, curl_easy_strerror(ec));
+                        ec, curl_easy_strerror(ec));
           res = ec;
           goto test_cleanup;
         }
@@ -108,7 +108,7 @@ static CURLcode test_lib677(const char *URL)
         }
         else if(ec) {
           curl_mfprintf(stderr, "curl_easy_recv() failed, with code %d (%s)\n",
-                        (int)ec, curl_easy_strerror(ec));
+                        ec, curl_easy_strerror(ec));
           res = ec;
           goto test_cleanup;
         }
index 24881039cdd31b4861cebe386ab74bbaf0091fcf..ea3f0268af01ef494a82c7303185bc91412d5083 100644 (file)
@@ -107,8 +107,8 @@ static CURLcode test_lib753(const char *URL)
 
   start_test_timing();
 
-  debug_config.nohex = 1;
-  debug_config.tracetime = 1;
+  debug_config.nohex = TRUE;
+  debug_config.tracetime = TRUE;
 
   curl_global_init(CURL_GLOBAL_DEFAULT);
 
index 2a46c3940d8294478b6041065d05e5b70c18f37f..b221f38cd0fadca8824e7d655f5a91169d61e692 100644 (file)
@@ -32,7 +32,7 @@ static int    known_offset; /* for test time tracing */
 
 void debug_dump(const char *timebuf, const char *text,
                 FILE *stream, const unsigned char *ptr,
-                size_t size, int nohex)
+                size_t size, bool nohex)
 {
   size_t i;
   size_t c;
@@ -221,7 +221,7 @@ int cli_debug_cb(CURL *handle, curl_infotype type,
     if(!traced_data) {
       if(!newl)
         log_line_start(output, idsbuf, type);
-      curl_mfprintf(output, "[%ld bytes data]\n", (long)size);
+      curl_mfprintf(output, "[%zu bytes data]\n", size);
       newl = 0;
       traced_data = 1;
     }
index d9eac5d0ebb57cf764a9134069f16a441a8de2f4..79d2f8529d96ac7e7355e02ef30f4b321274c5d9 100644 (file)
 
 void debug_dump(const char *timebuf, const char *text,
                 FILE *stream, const unsigned char *ptr,
-                size_t size, int nohex);
+                size_t size, bool nohex);
 
 struct libtest_trace_cfg {
-  int tracetime;  /* 0 represents FALSE, anything else TRUE */
-  int nohex;      /* 0 represents FALSE, anything else TRUE */
+  bool tracetime;
+  bool nohex;
 };
 
 extern struct libtest_trace_cfg debug_config;
index 6fac6dbc4b1930d12604ac6f928b7da5d99a8d61..35b38fd8b6255edf11900f04019631834f53fbf7 100644 (file)
@@ -39,8 +39,7 @@ static void splayprint(struct Curl_tree *t, int d, char output)
       printf("  ");
 
   if(output) {
-    printf("%ld.%ld[%d]", (long)t->key.tv_sec,
-           (long)t->key.tv_usec, i);
+    printf("%ld.%ld[%d]", (long)t->key.tv_sec, (long)t->key.tv_usec, i);
   }
 
   for(count = 0, node = t->samen; node != t; node = node->samen, count++)