At 10:50 AM 2/22/2003, Jim Jagielski wrote:
>Of course for 2.1, but also for 2.0 as well... Right now, SSLMutex
>is pretty binary :)

Only one observation below, otherwise ++1 for both 2.0 and 2.1 customers
(Jim and I recently spent a week fighting solaris 9 pthread mutexes, extra 
backup options are *always* goodness!)

>     else {
>         return apr_pstrcat(cmd->pool, "Invalid SSLMutex argument ", 
>-                           arg, NULL);
>+                           arg, " (", ssl_valid_ssl_mutex_string, ")", NULL);

the idea here was most cool, but I'd add to the "(" comment that

>+                           arg, " (options valid on this platform are: ",
>+                           ssl_valid_ssl_mutex_string, ")", NULL);

Bill 

Reply via email to