aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYang Tse <yangsita@gmail.com>2007-02-22 06:19:39 +0000
committerYang Tse <yangsita@gmail.com>2007-02-22 06:19:39 +0000
commit9bf9617ad6206f99d239f0a18c8f082add832152 (patch)
tree1646b0b379ec17d7165e3d41194030cd71c05ee3
parent69565afab0be081211df57a245a222cbd3e43ed3 (diff)
Fix compiler warnings
"case label value exceeds maximum value for type" and "comparison is always false due to limited range of data type" Both triggered when using a bool variable as the switch variable in a switch statement and using enums for the case targets.
-rw-r--r--lib/content_encoding.c12
-rw-r--r--lib/urldata.h12
2 files changed, 13 insertions, 11 deletions
diff --git a/lib/content_encoding.c b/lib/content_encoding.c
index 97f834177..a363f6db0 100644
--- a/lib/content_encoding.c
+++ b/lib/content_encoding.c
@@ -5,7 +5,7 @@
* | (__| |_| | _ <| |___
* \___|\___/|_| \_\_____|
*
- * Copyright (C) 1998 - 2006, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2007, 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
@@ -53,14 +53,6 @@
#define COMMENT 0x10 /* bit 4 set: file comment present */
#define RESERVED 0xE0 /* bits 5..7: reserved */
-enum zlibState {
- ZLIB_UNINIT, /* uninitialized */
- ZLIB_INIT, /* initialized */
- ZLIB_GZIP_HEADER, /* reading gzip header */
- ZLIB_GZIP_INFLATING, /* inflating gzip stream */
- ZLIB_INIT_GZIP /* initialized in transparent gzip mode */
-};
-
static CURLcode
process_zlib_error(struct connectdata *conn, z_stream *z)
{
@@ -76,7 +68,7 @@ process_zlib_error(struct connectdata *conn, z_stream *z)
}
static CURLcode
-exit_zlib(z_stream *z, bool *zlib_init, CURLcode result)
+exit_zlib(z_stream *z, zlibInitState *zlib_init, CURLcode result)
{
inflateEnd(z);
*zlib_init = ZLIB_UNINIT;
diff --git a/lib/urldata.h b/lib/urldata.h
index 3325699d7..6a1f7045e 100644
--- a/lib/urldata.h
+++ b/lib/urldata.h
@@ -521,6 +521,16 @@ struct hostname {
#define KEEP_WRITE_HOLD 8 /* when set, no writing should be done but there
might still be data to write */
+#ifdef HAVE_LIBZ
+typedef enum {
+ ZLIB_UNINIT, /* uninitialized */
+ ZLIB_INIT, /* initialized */
+ ZLIB_GZIP_HEADER, /* reading gzip header */
+ ZLIB_GZIP_INFLATING, /* inflating gzip stream */
+ ZLIB_INIT_GZIP /* initialized in transparent gzip mode */
+} zlibInitState;
+#endif
+
/*
* This struct is all the previously local variables from Curl_perform() moved
* to struct to allow the function to return and get re-invoked better without
@@ -581,7 +591,7 @@ struct Curl_transfer_keeper {
#define COMPRESS 3 /* Not handled, added for completeness */
#ifdef HAVE_LIBZ
- bool zlib_init; /* True if zlib already initialized;
+ zlibInitState zlib_init; /* possible zlib init state;
undefined if Content-Encoding header. */
z_stream z; /* State structure for zlib. */
#endif