<div dir="ltr"><div><div><div><div>Hi all,<br></div> the attached patch (a merge request from lp:~kinkie/squid/mempools-nozero) changes MEMPROXY_CLASS so that it doesn't zero memory before releasing it to requestors. The constructors of all classes using that feature were checked, so that they fully initialize all data members and thus require no zeroing.<br>It also changes Debug::OutStream to use MEMPROXY_CLASS instead of new/derelete, and changes constructors to initializer lists whenever possible and sensible (aka justified by other changes), same for dropping HERE and adopting nullptr.<br>It also changes some code in Makefile.am to use make variables in place of full paths.<br></div>The code has been run-tested (default config only), build-farm-tested and polygraph-tested. It shows a small but noticeable performance improvement in performance (of course, numbers are to be taken with a big grain of salt).<br><br></div>When reviewing, please do not focus on formatting issues; if this patch passes review I'll run a source-formatting before merging.<br><br></div>Thanks!<br clear="all"><div><div><div><div><div><div><br>-- <br><div class="gmail_signature"> Kinkie<br></div>
</div></div></div></div></div></div></div>