Re: [libmicrohttpd] 0.9.32 configure --disable-https; -Wsign-compare warnings

[Christian Grothoff]

Thanks for reporting. I've fixed this in SVN 31160 (both the compilation
issue and the
warnings).

Happy hacking!

Christian


On 12/06/2013 11:24 PM, Дилян Палаузов wrote:
> Hello,
> 
> when libmicrohttpd0932 is configured with --disable-https, the macro
> HTTPS_SUPPORT is undefined, and  struct MHD_Connection, defined in
> src/microhttpd/internal.h, does not contain the suspended and resuming
> fields.
> 
> Then src/microhttpd/connection.c : cleanup_connection : lines 2082-2094,
> does not compile, as it tries to access
> MHD_Connection.suspended/resuming, even when HTTPS_SUPPORT is undefined.
> 
> In addition, my compiler gcc (GCC) 4.7.4 20131123 (prerelease) warns about:
> 
> connection.c: In function ‘try_ready_normal_body’:
> connection.c:355:42: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
> connection.c:356:43: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
> connection.c:362:44: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
> connection.c: In function ‘try_ready_chunked_body’:
> connection.c:429:15: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
> connection.c:446:41: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
> connection.c:454:42: warning: comparison between signed and unsigned
> integer expressions [-Wsign-compare]
> 
> I use CFLAGS='-pipe -O3 -flto  -Wl,-S -Wl,--hash-style=gnu -Wl,-O1
> -Wl,-z,relro -Wall -Wextra' in config.site .
> 
> Kind regards
>   Dilyan Palauzov
>