bug-wget
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]