On Wednesday 19 September 2001 07:40 pm, Justin Erenkrantz wrote:
> Calling the AcceptMutex proc_pthread is just silly.  1.3 got it
> right by calling it pthread.
>
> This would also be one less thing for a person who uses this new
> feature in 1.3 to change when they upgrade to 2.0.
>
> Would anyone raise a fit if I committed this?  -- justin

Why is calling it proc_pthread silly?  We are talking about a pthread based
process lock.  Personally, I think Apache 1.3 should be changed, especially
since it hasn't been released yet.  My concern is that calling it a pthread lock
makes it sound like we are just locking threads.

Ryan

> Index: server/mpm_common.c
> ===================================================================
> RCS file: /home/cvs/httpd-2.0/server/mpm_common.c,v
> retrieving revision 1.67
> diff -u -r1.67 mpm_common.c
> --- server/mpm_common.c       2001/09/18 22:13:57     1.67
> +++ server/mpm_common.c       2001/09/20 02:29:39
> @@ -604,7 +604,7 @@
>      }
>  #endif
>  #if APR_HAS_PROC_PTHREAD_SERIALIZE
> -    else if (!strcasecmp(arg, "proc_pthread")) {
> +    else if (!strcasecmp(arg, "pthread")) {
>          ap_accept_lock_mech = APR_LOCK_PROC_PTHREAD;
>      }
>  #endif
> @@ -621,7 +621,7 @@
>                             ", sysvsem"
>  #endif
>  #if APR_HAS_PROC_PTHREAD_SERIALIZE
> -                           ", proc_pthread"
> +                           ", pthread"
>  #endif
>                             , NULL);
>      }

-- 

______________________________________________________________
Ryan Bloom                              [EMAIL PROTECTED]
Covalent Technologies                   [EMAIL PROTECTED]
--------------------------------------------------------------

Reply via email to