From 61ea058d9f9e5c81ce0ebfafcf7a0df4874a80d3 Mon Sep 17 00:00:00 2001 From: Yang Tse Date: Fri, 18 Sep 2009 14:48:16 +0000 Subject: fix compiler warning: comparison between signed and unsigned --- lib/setup.h | 6 ++++++ src/main.c | 12 +++++++++--- 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/lib/setup.h b/lib/setup.h index 43a322af5..f3ea36f8d 100644 --- a/lib/setup.h +++ b/lib/setup.h @@ -293,6 +293,7 @@ # define fstat(fdes,stp) _fstati64(fdes, stp) # define stat(fname,stp) _stati64(fname, stp) # define struct_stat struct _stati64 +# define LSEEK_ERROR (__int64)-1 #endif /* @@ -307,12 +308,17 @@ # define fstat(fdes,stp) _fstat(fdes, stp) # define stat(fname,stp) _stat(fname, stp) # define struct_stat struct _stat +# define LSEEK_ERROR (long)-1 #endif #ifndef struct_stat # define struct_stat struct stat #endif +#ifndef LSEEK_ERROR +# define LSEEK_ERROR (off_t)-1 +#endif + /* * Default sizeof(off_t) in case it hasn't been defined in config file. */ diff --git a/src/main.c b/src/main.c index d1152f9d8..cbe3ee5fe 100644 --- a/src/main.c +++ b/src/main.c @@ -218,6 +218,7 @@ typedef enum { # define fstat(fdes,stp) _fstati64(fdes, stp) # define stat(fname,stp) _stati64(fname, stp) # define struct_stat struct _stati64 +# define LSEEK_ERROR (__int64)-1 #endif /* @@ -232,12 +233,17 @@ typedef enum { # define fstat(fdes,stp) _fstat(fdes, stp) # define stat(fname,stp) _stat(fname, stp) # define struct_stat struct _stat +# define LSEEK_ERROR (long)-1 #endif #ifndef struct_stat # define struct_stat struct stat #endif +#ifndef LSEEK_ERROR +# define LSEEK_ERROR (off_t)-1 +#endif + /* * Default sizeof(off_t) in case it hasn't been defined in config file. */ @@ -3301,13 +3307,13 @@ static int my_seek(void *stream, curl_off_t offset, int whence) /* this code path doesn't support other types */ return 1; - if(-1 == lseek(in->fd, 0, SEEK_SET)) + if(LSEEK_ERROR == lseek(in->fd, 0, SEEK_SET)) /* couldn't rewind to beginning */ return 1; while(left) { long step = (left>MAX_SEEK ? MAX_SEEK : (long)left); - if(-1 == lseek(in->fd, step, SEEK_CUR)) + if(LSEEK_ERROR == lseek(in->fd, step, SEEK_CUR)) /* couldn't seek forwards the desired amount */ return 1; left -= step; @@ -3315,7 +3321,7 @@ static int my_seek(void *stream, curl_off_t offset, int whence) return 0; } #endif - if(-1 == lseek(in->fd, offset, whence)) + if(LSEEK_ERROR == lseek(in->fd, offset, whence)) /* couldn't rewind, the reason is in errno but errno is just not portable enough and we don't actually care that much why we failed. We'll let libcurl know that it may try other means if it wants to. */ -- cgit v1.2.3