Re: [PATCH] http: fix some printf format warnings on 32-bit builds

On Tue, Nov 10, 2015 at 5:22 PM, Eric Sunshine [off-list ref] wrote:

On Tue, Nov 10, 2015 at 7:23 PM, Ramsay Jones
[off-list ref] wrote:

quoted

Commit f8117f55 ("http: use off_t to store partial file size",
02-11-2015) changed the type of some variables from long to off_t.
The 32-bit build, which enables the large filesystem interface
(_FILE_OFFSET_BITS == 64), defines the off_t type as a 64-bit
integer, whereas long is a 32-bit integer. This results in a couple
of printf format warnings.

In order to suppress the warnings, change the format specifier to use
the PRIuMAX macro and cast the off_t argument to uintmax_t. (See also
the http_opt_request_remainder() function, which uses the same
solution).

I just ran across the problem when building 'next' on my Mac and was
about to investigate, so am happy to find that the work has already
been done. Thanks.

My machine is 64-bit, though, so perhaps it's misleading to
characterize this as a fix for 32-bit builds. In particular, off_t is
'long long' on this machine, so it complains about the "long" format
specifier.

+Lars

I wonder if 32 bit compilation can be part of travis.