<p dir="ltr">Looks reasonable to me, except that new class names do not comply with naming conventions</p>
<p dir="ltr">-- <br>
  Francesco</p>
<div class="gmail_quote">On Apr 20, 2016 8:00 AM, "Amos Jeffries" <<a href="mailto:squid3@treenet.co.nz">squid3@treenet.co.nz</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 12/04/2016 11:31 p.m., Amos Jeffries wrote:<br>
> This is the first of the followup patches I promised when applying the<br>
> un-polished bug 4438 patch to 4.0.8.<br>
><br>
><br>
> Convert all the objects using the libmem "old API" for as-needed pools<br>
> to using the MEMPROXY_CLASS() API which is better designed for late<br>
> initialization.<br>
><br>
> The GetPool(type) function is now guaranteed to produce a<br>
> pre-initialized pool for all valid pool types. Except in the case where<br>
> it is recursively being used by Mem::Init() to perform that initialization.<br>
><br>
<br>
This is about to pass the 10-day hold criteria for merge. If there are<br>
no objections I will apply it in a day or two after todays release has<br>
settled.<br>
<br>
Amos<br>
<br>
_______________________________________________<br>
squid-dev mailing list<br>
<a href="mailto:squid-dev@lists.squid-cache.org">squid-dev@lists.squid-cache.org</a><br>
<a href="http://lists.squid-cache.org/listinfo/squid-dev" rel="noreferrer" target="_blank">http://lists.squid-cache.org/listinfo/squid-dev</a><br>
</blockquote></div>