> On Mon, 1 Apr 2002, Bill Stoddard wrote: > > > I suspect apr_bucket_alloc_create() is called in the wrong place. > > Working on it. SHould have a patch in 30 min or so. > > A semi-related question:: > > Should apr_bucket_alloc_create() be registering a cleanup on the pool > that's passed into it so that apr_bucket_alloc_destroy() is called > automatically when that pool dies?
+1, yes. > I didn't do this before because I was > hoping to get rid of the pool parameter to apr_bucket_alloc_create() > entirely. Err, ok, you could do that, but I'd be in favor of having pools do our cleanup. When we need to shutdown*, the allocator should be destroyed and memory should be returned. *) shutdown, or any other operation that will tear down stuff. > Right now we're not using it for anything. Only for the bucket_alloc struct itself, right? > --Cliff