The 22 is a magic number, based in the fact that 1e21 > INT64_MAX I think it should be #defined in wget.h It could be defined variable depending on the different MAX values, eg. sizeof( STRINGIZE( IN
Pre-POSIX compatibility that Wget used to do would probably be inappropriate in this age, but it would be nice if Wget kept supporting C89. High portability was always part of Wget's mission, and tha
Hey cool, you made these patches so satisfy MS Visual C ??? But if you are in the mood to fix warnings, here are some more ;-) CFLAGS="-O2 -std=gnu99 -pedantic -Wall -Wstrict-prototypes -Wold-stylede
Plus a chunk of small changes removing mixed code and declarations. Attachment: 0001-Removed-mixed-declarations-and-code-making-more-C89-.patch Description: Text Data
I tried it, and it only needs a one line change to be compilable :) Remember to run CC='gcc -std=c89' ./configure first, or otherwise you're using a configuration tailed to another compiler with a c8
Taking this into account: is there any good point in not using long long ? The code is already "polluted" by c99 stuff. Right. I figure "someone" needs to decide where the compatibility line should b
as David Ryskalczyk stated, just two printf format specifiers might cause the havoc. I think, there is not need to use wgint instead of off_t. @Guiseppe: please apply the appended patches (maybe addi
lovely. I am going to push it soon with some small adjustments. Thanks for the great work. Whenever it happens to be in the same place, I'll buy you a beer :-) Cheers, Giuseppe
And use semicolons to separate multiple phrases in a description. -- src/main.c | 280 ++++++++++++++++++++++++++++++-- 1 files changed, 140 insertions(+), 140 deletions(-) diff --git a/src/main.c b/s
Hi Gisle, please always answer to the mailing list. You don't have the latest git version, at least your diff isn't based on it. I pushed the C89 changes right before I asked you to give me the compi
I noticed that the gnulib modules ftello, mkstemp and strtok_r are needed for windows build, but not listed in bootstrap.conf Patch attached. These are the errors that I got without it: Attachment: w
Hi, I've recently undertaken an exercise to map the command line switches of WGET 1.15 alongside the commands available in the config file I specify with --config=FILE. See attached file for detail.
wget --warc-file=httpbin -qO- https://httpbin.org/get How to convert the warc format to the actual header of requests and responses? Greetings WARC is gzipped plain text. wget --warc-file=httpbin --n
The make fails[...] I've redone the installation process from the very scratch: * ran bzr branch in a newly created folder, * ran bootstrap script. The bootstrap script told warnings: on stdout: Noti
Hi, attached you can find a patch that proposes a change to the file warc.c. The change will use url_escape to escape reserved characters in the redirect_location. Up to the current version (1.19) wg