diff options
author | Yang Tse <yangsita@gmail.com> | 2008-10-24 01:27:00 +0000 |
---|---|---|
committer | Yang Tse <yangsita@gmail.com> | 2008-10-24 01:27:00 +0000 |
commit | 6ea91af2f8f787f16b1bde88ae2293adcc961986 (patch) | |
tree | b7a84538b1b6217e877a01a2b41367d330d2e712 | |
parent | b7674403995be67dae5017b9dd573cc7b4e65825 (diff) |
fix compiler warning
-rw-r--r-- | lib/http.c | 7 | ||||
-rw-r--r-- | lib/http_chunks.c | 6 | ||||
-rw-r--r-- | src/getpass.c | 4 | ||||
-rw-r--r-- | src/main.c | 12 |
4 files changed, 13 insertions, 16 deletions
diff --git a/lib/http.c b/lib/http.c index 068748c3c..b6609c818 100644 --- a/lib/http.c +++ b/lib/http.c @@ -1836,13 +1836,10 @@ static CURLcode https_connecting(struct connectdata *conn, bool *done) /* perform SSL initialization for this socket */ result = Curl_ssl_connect_nonblocking(conn, FIRSTSOCKET, done); - if(result) { + if(result) conn->bits.close = TRUE; /* a failed connection is marked for closure to prevent (bad) re-use or similar */ - return result; - } - - return CURLE_OK; + return result; } #ifdef USE_SSLEAY diff --git a/lib/http_chunks.c b/lib/http_chunks.c index e0887483a..630a5a265 100644 --- a/lib/http_chunks.c +++ b/lib/http_chunks.c @@ -85,9 +85,9 @@ We avoid the use of isxdigit to accommodate non-ASCII hosts. */ static bool Curl_isxdigit(char digit) { - return (digit >= 0x30 && digit <= 0x39) /* 0-9 */ - || (digit >= 0x41 && digit <= 0x46) /* A-F */ - || (digit >= 0x61 && digit <= 0x66); /* a-f */ + return (bool)( (digit >= 0x30 && digit <= 0x39) /* 0-9 */ + || (digit >= 0x41 && digit <= 0x46) /* A-F */ + || (digit >= 0x61 && digit <= 0x66) ); /* a-f */ } void Curl_httpchunk_init(struct connectdata *conn) diff --git a/src/getpass.c b/src/getpass.c index 7209a2ab1..8d20eb48c 100644 --- a/src/getpass.c +++ b/src/getpass.c @@ -5,7 +5,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2005, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 1998 - 2008, Daniel Stenberg, <daniel@haxx.se>, et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -113,7 +113,7 @@ char *getpass_r(const char *prompt, char *buffer, size_t buflen) fputs(prompt, stderr); for(i=0; i<buflen; i++) { - buffer[i] = getch(); + buffer[i] = (char)getch(); if ( buffer[i] == '\r' || buffer[i] == '\n' ) { buffer[i] = 0; break; diff --git a/src/main.c b/src/main.c index e12c26ed5..1e06ef067 100644 --- a/src/main.c +++ b/src/main.c @@ -1852,7 +1852,7 @@ static ParameterError getparameter(char *flag, /* f or -long-flag */ config->disable_epsv = toggle; break; case 'E': /* --epsv */ - config->disable_epsv = !toggle; + config->disable_epsv = (bool)(!toggle); break; #ifdef USE_ENVIRONMENT case 'f': @@ -2021,7 +2021,7 @@ static ParameterError getparameter(char *flag, /* f or -long-flag */ config->disable_eprt = toggle; break; case 'Z': /* --eprt */ - config->disable_eprt = !toggle; + config->disable_eprt = (bool)(!toggle); break; default: /* the URL! */ @@ -2148,7 +2148,7 @@ static ParameterError getparameter(char *flag, /* f or -long-flag */ config->ftp_ssl_reqd = toggle; break; case 'w': /* --no-sessionid */ - config->disable_sessionid = !toggle; + config->disable_sessionid = (bool)(!toggle); break; case 'x': /* --ftp-ssl-control */ config->ftp_ssl_control = toggle; @@ -2172,7 +2172,7 @@ static ParameterError getparameter(char *flag, /* f or -long-flag */ config->post301 = toggle; break; case '1': /* --no-keepalive */ - config->nokeepalive = !toggle; + config->nokeepalive = (bool)(!toggle); break; case '3': /* --keepalive-time */ if(str2num(&config->alivetime, nextarg)) @@ -2602,7 +2602,7 @@ static ParameterError getparameter(char *flag, /* f or -long-flag */ break; case 'N': /* disable the output I/O buffering */ - config->nobuffer = !toggle; + config->nobuffer = (bool)(!toggle); break; case 'O': /* --remote-name */ if(subletter == 'a') { /* --remote-name-all */ @@ -2727,7 +2727,7 @@ static ParameterError getparameter(char *flag, /* f or -long-flag */ config->mute = config->noprogress = TRUE; else config->mute = config->noprogress = FALSE; - config->showerror = !toggle; /* toggle off */ + config->showerror = (bool)(!toggle); /* toggle off */ break; case 'S': /* show errors */ |