On Wed, 11 Apr 2001, dean gaudet wrote:

> On Wed, 11 Apr 2001 [EMAIL PROTECTED] wrote:
>
> > On Wed, 11 Apr 2001, Harrie Hazewinkel wrote:
> >
> > > + /* The MPM types below should actually a string which enables better 
>identification */
> > > + #define AP_MPMQ_BEOS               1    /* MPM:Beos                   */
> > > + #define AP_MPMQ_PERCHILD           2    /* MPM:perchild               */
> > > + #define AP_MPMQ_PREFORK            3    /* MPM:prefork                */
> > > + #define AP_MPMQ_SPMT_OS2           4    /* MPM:spmt_os2               */
> > > + #define AP_MPMQ_WINNT              5    /* MPM:winnt                  */
> > > + #define AP_MPMQ_THREADED           6    /* MPM:threaded               */
> >
> > I dislike this.  The Modules shouldn't care which MPM they are running on.
> > Why do we need this option?
>
> not to mention that it should be a string based query instead of
> hard-coded integers... otherwise it requires patching every time a new MPM
> is added, which is wrong.

I agree.  Of course, a string would require a new query function, because
the current one always returns an int.  Or, we could abstract it out to
return a void *.

Ryan

_______________________________________________________________________________
Ryan Bloom                              [EMAIL PROTECTED]
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------

Reply via email to