[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-wget] Segmentation fault with current development version of wg
From: |
Giuseppe Scrivano |
Subject: |
Re: [Bug-wget] Segmentation fault with current development version of wget |
Date: |
Sun, 16 Jun 2013 22:35:11 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Thanks, the code seems correct but since you are going to write a lot of
code during your Summer of Code project, I will be very pedantic. This
time I have just amended the changes before push your patch.
Darshit Shah <address@hidden> writes:
> Please be aware that Wget needs to know the size of the POST data in
> -advance. Therefore the argument to @code{--post-file} must be a regular
we put two empty spaces after a sentence. I have fixed this change in
other places too.
> + switch (statcode)
> + {
I have indented the '{' correctly. Two empty spaces. Same for the
closing curly bracket.
> + if ((!(*dt & RETROKF) && !opt.content_on_error) || head_only ||
> (opt.method && strcasecmp (opt.method, "GET") != 0))
long line, I have splitted it.
--
Giuseppe