Pier wrote:
> The only thing I don't like (much) is the name of the prototype functions:
> - warp_socket_pool_create
> - warp_socket_pool_acquire_socket
> - warp_socket_pool_return_socket
>
> And a lack of warp_socket_pool_destroy...
>
> I would have called them
>
> - warp_sockpool_create
> - warp_sockpool_acquire
> - warp_sockpool_release
> - warp_sockpool_destroy
>
I'll change the function names and add a destroy function. For the destroy function
I'm assuming you want:
apr_status_t warp_sockpool_destroy(warp_socket_pool * pool)
{
return apr_thread_mutex_destroy(pool->pool_mutex);
}
Is this right? Am I missing something?
Simon
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
