It's a bit confusing to have Apache::MPM_IS_THREADED and Apache::MPMQ_IS_THREADED, the first being the run_time value, the second just the key you pass to the query. But I guess with a solid documentation this should be fine.
I don't mind changing this to Apache::MPM::is_threaded(), I just didn't want to change too much. it probably makes more sense as a function than a constsub anyway...
I think I like Apache::MPM::is_threaded() better than Apache::MPM_IS_THREADED (as a faster alias to
Apache::MPM::mpm_query(MPMQ_IS_THREADED), since it's preset at start time)
Or should it be:
Apache->is_threaded()? like Apache->server? which is another constant, or Apache->request...
On the other hand, using a constant is probably better for efficiency and smaller memory footprint for modules that do some conditional code based on this flag, I see no reason why Apache::MPM::is_threaded can't be a constsub.
__________________________________________________________________ Stas Bekman JAm_pH ------> Just Another mod_perl Hacker http://stason.org/ mod_perl Guide ---> http://perl.apache.org mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com http://modperlbook.org http://apache.org http://ticketmaster.com
-- Reporting bugs: http://perl.apache.org/bugs/ Mail list info: http://perl.apache.org/maillist/modperl.html
