On 1/24/22 9:42 AM, Ruediger Pluem wrote:
>
>
> On 1/20/22 5:14 PM, Yann Ylavic wrote:
>> Index: server/util.c
>> ===================================================================
>> --- server/util.c (revision 1897156)
>> +++ server/util.c (working copy)
>> @@ -3261,6 +3261,56 @@ AP_DECLARE(void *) ap_realloc(void *ptr, size_t si
>> return p;
>> }
>>
>> +#if defined(AP_THREAD_LOCAL) && !APR_VERSION_AT_LEAST(1,8,0)
>> +struct thread_ctx {
>> + apr_thread_start_t func;
>> + void *data;
>> +};
>> +
>> +static AP_THREAD_LOCAL apr_thread_t *current_thread;
>
> Shouldn't we do
>
> static AP_THREAD_LOCAL apr_thread_t *current_thread = NULL;
>
> for the sake of clarity?
Otherwise looks good.
Regards
Rüdiger
