<div dir="ltr">Merged with requested changes as r14244.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 24, 2015 at 7:10 PM, Amos Jeffries <span dir="ltr"><<a href="mailto:squid3@treenet.co.nz" target="_blank">squid3@treenet.co.nz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 25/08/2015 3:36 a.m., Kinkie wrote:<br>
> Hi,<br>
>   part 4 (of 5) patches from the mempools-nozero branch: debug<br>
><br>
> This patch transforms Debug::OutStream into a MEMPROXY_CLASS instead of<br>
> rolling its own new and delete operators. The biggest changes come from<br>
> pulling in various Makefile.am files the necessary dependencies.<br>
> I've taken the chance to also rely more on Makefile variables in various<br>
> file-copying recipes in order to simplify them a bit.<br>
><br>
> Patch with big context attached.<br>
><br>
<br>
</span>in src/icmp/Makefile.am:<br>
* "#libmem is needed for"looks like it should be up on COPIED_SOURCE<br>
 - I see no libmem/libmisc file reference in the list below it<br>
 - the libmisc is also MIA<br>
<br>
<br>
in test-suite/Makefile.am:<br>
* I woud rather the STUBS= list was one file per line now if its going<br>
to be wrapped at all.<br>
<br>
<br>
in tools/Makefile.am<br>
* please use alphabetical order on cachemgr__CGIEXT__SOURCES changes<br>
 - that will even cut the patch by 1 line.<br>
<br>
+1. with the above<br>
<br>
Amos<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><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">    Francesco</div>
</div>