apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Cliff Woolley <jwool...@virginia.edu>
Subject Re: [PATCH] another bucket allocator implementation
Date Sun, 31 Mar 2002 20:37:40 GMT
On Sun, 31 Mar 2002, Brian Pane wrote:

>                       requests/second
>                       ---------------
>  Test        2.0.34      2.0.34 +           2.0.34 +
>  file        baseline    Brian's patch    Cliff's patch
> 50k.shtml      654           667              686
> incl.shtml    1870          1992             2007

Sweet!!  In both cases this is really great news.

PS: I remembered at lunch that there was a buglet in
apr_bucket_alloc_destroy() I'd forgotten to fix.  Since list itself is
allocated from list->allocator, we need to save a pointer to the allocator
before we call apr_allocator_free() so that we will still have a valid
pointer to pass to apr_allocator_destroy() afterward.  Attached is a
revised patch.

--Cliff


--------------------------------------------------------------
   Cliff Woolley
   cliffwoolley@yahoo.com
   Charlottesville, VA


Mime
View raw message