Date: Mon, 12 Jul 1999 11:55:05 -0700 (PDT)
From: Dean Gaudet <dgaudet@arctic.org>
To: new-httpd@apache.org
Subject: Re: cvs commit: apache-2.0/mpm/src/main http_config.c http_main.c
In-Reply-To: <378A36A5.DF0292CC@algroup.co.uk>
Message-ID: <Pine.LNX.3.96dg4.990712115324.20623d-100000@twinlark.arctic.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: new-httpd-owner@apache.org
Precedence: bulk
Reply-To: new-httpd@apache.org
Status: O


On Mon, 12 Jul 1999, Ben Laurie wrote:

> Dean Gaudet wrote:
> > 
> > On Sun, 11 Jul 1999, Ben Laurie wrote:
> > 
> > > Well, if I'm going to preserve the original behaviour as much as I can,
> > > yes. The reason being that hooks are defined early on, but I need
> > > -DSHOW_HOOKS before they are, but there's also a pre-command-line hook
> > > that has to come _after_ hooks have been hooked (obviously).
> > 
> > Maybe the pre_command_line hook shouldn't be there.  I forget exactly why
> > I wanted it, I think in many ways what I wanted was a hook to initialize
> > modules just after they first load... in case they have any memory that
> > needs initializing.
> 
> You use it in prefork.c (I think) ... if that's all it's for, you could
> do it in register_hooks?

Yeah, it just calls INIT_SIGLIST().  Totally -- if register_hooks is
guaranteed to be called just once for each module as its loaded, before
anything else in the module is called, then that's just about exactly what
I was thinking of for pre_command_line.  So just do a name change :) 

Dean