On Wed 25 Apr 2001, Andrew M. Bishop wrote:
> Paul Slootman <[EMAIL PROTECTED]> wrote:
> > A couple of problems:
> >
> > - There is a blank line before the .TH line in wwwoffled.man which
> > causes (at least) the debian man program to spew out a blank
> > page at the beginning.
>
> That file has been like that for ~20 months, I am surprised that
> nobody has noticed it before.
Hmm, maybe I should read the manpages more often :-)
I saw it because I was looking for a way to prevent it forking into the
background to debug the other problem.
> I guess this shows that most people must use the upgrade-config.pl
> script or I imagine that more people would have reported this bug.
Agreed.
> > Perhaps the result of reading in the config file should be
> > sanity-checked before proceeding?
>
> If you look at the other options that use ConfigItemFoo->nitems they
> are all protected by testing that ConfigItemFoo is not-NULL. By some
> oversight I missed it when I added these two.
Yes, but then you perform the check each time an URL is accessed,
whereas you really only need to do it once after reading the config.
However, I realize that would take a lot of effort at this point in the
development of wwwoffle.
> All of these bugs will be fixed in the next version.
Thanks!
Paul Slootman
