Re: url.c (in_url_list_p): why bool verbose?

[Josh Williams]

On 8/22/07, Josh Williams <[EMAIL PROTECTED]> wrote:
> In src/url.c, function in_url_list_p, there is an  argument called
> "bool verbose", but it is never used. Furthermore, the verbose option
> is defined in our options struct.
>
> Should this argument be removed?

Below is a patch of this change.

Index: src/spider.c
===================================================================
--- src/spider.c        (revision 2336)
+++ src/spider.c        (working copy)
@@ -67,7 +67,7 @@
 };

 static bool
-in_url_list_p (const struct url_list *list, const char *url, bool verbose)
+in_url_list_p (const struct url_list *list, const char *url)
 {
   const struct url_list *ptr;

@@ -100,7 +100,7 @@
       list->url = referrer ? xstrdup (referrer) : NULL;
       hash_table_put (visited_urls_hash, xstrdup (url), list);
     }
-  else if (referrer && !in_url_list_p (list, referrer, false))
+  else if (referrer && !in_url_list_p (list, referrer))
     {
       /* Append referrer at the end of the list */
       struct url_list *newnode;