On Wed, Mar 25, 2009 at 01:21:25AM +0100, Joerg Sonnenberger wrote:
On Wed, Mar 25, 2009 at 12:03:05AM +0000, corvid wrote:
Jorge wrote:
Skimming the code it looks like the cast to long for int is the way to go. If a:
- URL_DATA(url)->len, content_type->str, + (long)URL_DATA(url)->len, content_type->str,
solves it. I'd commit it and let the explanation come after the patch.
I was thinking of adding long casts for the Content-Length headers in file.c as well. Sound good?
Shouldn't that be (s)size_t?
The parameter is specified as %ld, so long seems OK. Now the number of parameters is relevant (quite irregular).
At least one vendor was smart enough to produce a 64bit platform with long != ssize_t and support for files larger than 32bit sounds highly useful...
I'd like to have the result of the suggested test. -- Cheers Jorge.-