On Fri, Aug 24, 2001 at 05:59:58AM -0000, [EMAIL PROTECTED] wrote:
> jerenkrantz    01/08/23 22:59:58
> 
>   Modified:    flood    CHANGES flood_farm.c
>   Log:
>   Okay, here's the "real" commit log for the fork()-based implementation of
>   flood.
>   
>   - Remove FLOOD_USE_THREADS as we have no way of determining if we have
>     threads other than APR.  (We'd say that threads were always enabled.)
>     I feel comfortable using APR's thread detection rather than mucking
>     with parameters ourselves.
>   - Add strtoq conversion for FreeBSD that #defines it to strtoll.  This
>     is a tad cheesy.  Thoughts?  I don't want to have #ifdef HAVE_STRTOLL
>     and #ifdef HAVE_STRTOQ all over the place.  Smells like an APR function.

does linux have strtoll? IIRC, we're just reading it into an 'int' anyway,
so why the long long?

>   - Remove forking stuff in flood.c as it doesn't make sense to fork() here
>     anyway as this is only bootstrap code.
>   - Make reports output either the current thread id or the pid

I assume this is only temporary until we have a better GUID? We really
shouldn't be calling apr_os_thread_id() and then trying to print it as
an int. How about just a counter for each new thread/process?

>   - Don't do OpenSSL locking when we aren't threaded

cool.

>   - Not all OpenSSL versions (FreeBSD 4.2 has 0.9.5a) has AUTO_RETRY.  So,
>     only set that option if we have that symbol.
>   
>   So, we now support FreeBSD.  Go forth.

very cool.

-aaron

Reply via email to