httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ruediger Pluem <rpl...@apache.org>
Subject Re: svn commit: r543515 - in /httpd/httpd/trunk: CHANGES modules/cache/mod_mem_cache.c
Date Fri, 01 Jun 2007 21:43:34 GMT


On 06/01/2007 11:18 PM, Eric Covener wrote:
> On 6/1/07, Ruediger Pluem <rpluem@apache.org> wrote:
> 
>> Ahh. Should have read
>> http://issues.apache.org/bugzilla/show_bug.cgi?id=41551#c2
>> before which answers my question :-).
>> Anyway another question: From a first glance your original patch and
>> this patch basicly seem to do the same thing.
>> But the original patch was said to be non working by the reporter whereas
>> this worked. Any idea why?
> 
> 
> A mystery to me as well, the one-sided endorsement from the reporter

Good. I feared I just missed something obvious :-).

> as well as Davi having contributed the original heap-to-pools patch
> was enough to sway me to use his patch instead.
> 

The advantage of his patch is that once his proposed apr_table_clone
is available in apr we can use this and things where we call it remain
more readable IMHO than with the apr_table_do approach (not saying that
your patch was unreadable).

Regards

RĂ¼diger

Mime
View raw message