]> granicus.if.org Git - curl/commitdiff
Fix compiler warnings:
authorYang Tse <yangsita@gmail.com>
Tue, 2 Feb 2010 16:21:37 +0000 (16:21 +0000)
committerYang Tse <yangsita@gmail.com>
Tue, 2 Feb 2010 16:21:37 +0000 (16:21 +0000)
(1) conversion from 'const int ' to 'unsigned char ', possible loss of data
(2) conditional expression is constant

src/main.c

index 0410b647180f27ed8919e8d3ee79227d1be68fda..c5f904f1329fc76cd4247ca40b6e271e62369a6f 100644 (file)
@@ -2300,7 +2300,7 @@ static ParameterError getparameter(char *flag, /* f or -long-flag */
       }
       break;
     case '#': /* --progress-bar */
-      config->progressmode = toggle?CURL_PROGRESS_BAR:0;
+      config->progressmode = (bool)(toggle?CURL_PROGRESS_BAR:0);
       break;
     case '0':
       /* HTTP version 1.0 */
@@ -4137,7 +4137,7 @@ header_callback(void *ptr, size_t size, size_t nmemb, void *stream)
 
     /* look for the 'filename=' parameter
        (encoded filenames (*=) are not supported) */
-    while (1) {
+    for(;;) {
       char *filename;
 
       while (*p && (p < end) && !ISALPHA(*p))
@@ -5170,7 +5170,7 @@ operate(struct Configurable *config, int argc, argv_item_t argv[])
 
         retrystart = cutil_tvnow();
 
-        do {
+        for(;;) {
           res = curl_easy_perform(curl);
           if (!curl_slist_append(easycode, "ret = curl_easy_perform(hnd);")) {
             res = CURLE_OUT_OF_MEMORY;
@@ -5292,7 +5292,7 @@ operate(struct Configurable *config, int argc, argv_item_t argv[])
           retry_sleep = retry_sleep_default;
           break;
 
-        } while(1);
+        }
 
         if((config->progressmode == CURL_PROGRESS_BAR) &&
            progressbar.calls)