>-----Original Message----- >From: Joe Orton [mailto:[EMAIL PROTECTED] [SNIP] > >On Mon, Mar 29, 2004 at 11:58:46AM -0800, Mathihalli, Madhusudan wrote: >> Sounds good - but you still need to delete the last_e. > >This is what I asked before - why? The apr_brigade_destroy(b) call >deletes the EOC bucket along with all the others a few lines further on >AFAICT.
That's right - the brigade_destroy() should delete the bucket also ! It's not required - the patch looks good as-is. -Madhu
