Re: [Bug-wget] config patch

[Giuseppe Scrivano]

vivi <address@hidden> writes:

> === modified file 'src/ChangeLog'
> --- src/ChangeLog     2010-08-01 20:55:53 +0000
> +++ src/ChangeLog     2010-08-05 17:55:28 +0000
> @@ -1,3 +1,18 @@
> +2010-08-04  Reza Snowdon <address@hidden>
> +
> +       * main.c (main): inserted 'defaults'
> +       * init.c: Include stdbool.h.
> +          (commands):Added config detailsi.

Typo here.


The patch seems fine, except I have noticed some trailing whitespaces
and no space between the function name and '('.  Can you please check
them?

The new option should be documented, can you do it in the doc/wget.texi
file?

Also, it is desiderable to test this new feature, if you like you can
add a new test under the tests/ directory.  I think it will be enough to
feed wget with an incorrect configuration file and check that wget exits
with an error.

Can you please do a "bzr rebase" on your patch before re-send it?

Thanks,
Giuseppe