httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Stoddard" <redd...@attglobal.net>
Subject Re: ap_brigade_destroy
Date Tue, 15 Aug 2000 02:23:50 GMT
I was recalling code from long ago... I was suprised that ap_register_cleanup, et. al now supported
clean-ups that return apr_status_t. I will not make this change.

Bill

>
> I haven't even thought about actually reviewing this yet, but the current
> API for cleanups is:
>
> apr_status_t func_name(void *data);
>
> From your comments it sounds like we have a function that looks like that,
> and you want to change it to void *func_name(void *data) so that it is a
> true cleanup.  What am I missing?
>
> Ryan
>
> On Mon, 14 Aug 2000, Bill Stoddard wrote:
>
> >
> > ap_brigade_destroy(void *data) should be returning a void * rather than apr_status_t.
This
> > will also save a cast to void * on the function when calling ap_register_cleanup.
> >
> > I will change the bucket brigades cleanups to return void * if I hear no objections.
> >
> > Bill
> >
> >
>
>
> _______________________________________________________________________________
> Ryan Bloom                        rbb@apache.org
> 406 29th St.
> San Francisco, CA 94131
> -------------------------------------------------------------------------------
>


Mime
View raw message