Dan Harkless <[EMAIL PROTECTED]> writes:
> > --- src/recur.c Sun Dec 17 20:28:20 2000
> > +++ src/recur.c.new Sun Mar 25 20:25:12 2001
> > @@ -165,7 +165,18 @@
> > first_time = 0;
> > }
> > else
> > + {
> > + u = newurl ();
> > + err = parseurl (this_url, u, 0);
> > + if (err == URLOK)
> > + {
> > + ulist = add_slist (ulist, u->url, 0); /* ??? */
> > + urls_downloaded = add_url (urls_downloaded, u->url, file);
> > + urls_html = add_slist (urls_html, file, NOSORT);
> > + }
> > + freeurl(u, 1);
> > ++depth;
> > + }
> >
> > if (opt.reclevel != INFINITE_RECURSION && depth > opt.reclevel)
> > /* We've exceeded the maximum recursion depth specified by the user. */
>
> Thanks for the patch, Carsten. Hopefully someone will have time to review
> it soon and apply it if it's appropriate.
>
> Does this look like the right fix, Hrvoje?
I'm not completely sure. I suspect that it doesn't handle the HTML
files that are downloaded but not followed recursively.
To be sure that *all* HTML files are handled, I think the addition
needs to be triggered from within retrieve_url, say by calling a
"register_html_file_for_conversion()". I think I'll provide such a
fix tonight.
On a side note, I think it might be more useful to remove the
first-step conversion as it seems to do cause more confusion than it
benefits.
