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
-------------------------------------------------------------------------------