On Wed, Oct 7, 2015 at 8:45 PM, Ruediger Pluem <rpl...@apache.org> wrote:
>
>
> On 10/07/2015 12:23 AM, yla...@apache.org wrote:
>>
>>  static void eor_bucket_destroy(void *data)
>>  {
>> -    request_rec *r = (request_rec *)data;
>> +    apr_bucket *h = data;
>
> IMHO this should be
>
> apr_bucket_refcount *h;

Good catch!

Actually, we even need:

typedef struct {
    apr_bucket_refcount refcount;
    request_rec *data;
} apr_bucket_eor;

and use this type everywhere for h.

Thanks,
Yann.

Reply via email to