Re: [PATCH] Restore mempools functionality for MemBlob

On 30/03/11 23:34, Kinkie wrote:
> Hi all,
> the attached patch increases the flexibility of mempools in
> supporting strings, and as a consequence enables MemBlob to use
> MemPools.
> Meant for trunk and 3.2, it passes the test-suite.
>

src/mem.cc: hunk @229
- if (size <= StrPoolsAttrs[i].obj_size) {
+ if (size > StrPoolsAttrs[i].obj_size)
+ break;

... relies on the pools being listed by size and scanned largest to
smallest. For the current ordering of StrPoolsAttrs with i++ the loop
will match or break out after only the first entry.

Amos

-- 
Please be using
   Current Stable Squid 2.7.STABLE9 or 3.1.11
   Beta testers wanted for 3.2.0.5