Return-Path: Delivered-To: apmail-new-httpd-archive@apache.org Received: (qmail 6358 invoked by uid 500); 15 Aug 2000 01:51:41 -0000 Mailing-List: contact new-httpd-help@apache.org; run by ezmlm Precedence: bulk X-No-Archive: yes Reply-To: new-httpd@apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list new-httpd@apache.org Received: (qmail 6347 invoked from network); 15 Aug 2000 01:51:41 -0000 X-Authentication-Warning: koj.rkbloom.net: rbb owned process doing -bs Date: Mon, 14 Aug 2000 18:56:34 -0700 (PDT) From: rbb@covalent.net X-Sender: rbb@koj.rkbloom.net To: new-httpd@apache.org Subject: Re: ap_brigade_destroy In-Reply-To: <009701c00623$19eba6f0$c1e01b09@raleigh.ibm.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Spam-Rating: locus.apache.org 1.6.2 0/1000/N 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 -------------------------------------------------------------------------------