<div dir="ltr"><div><div><div><div><div><div>I'm going over HttpHeader to see if there's any possible improvements to be obtained by dragging it (kicking and screaming, from the look of it) into the 10's.<br><br></div>Here's a list of weirdnesses I'm coming to find, hopefully someone has any idea of WHY they are like this, or if they can be safely be removed:<br><br>- HttpHeader::insertTime makes no sense. It's O(N) and it's always used in two patterns: delById(HDR); insertTime(HDR) and if (!has(HDR) insertTime(HDR)).<br>I've replaced with putTime everywhere<br></div>- why doesn't HttpHeaderEntry use a copy-constructor, relying instead in a clone() method?<br></div>- why the strange loop in HttpHeader's copy-constructor? Replaced with a "= default", it should be just fine..<br></div>- statistics accounting: we count A LOT (see HttpHeader::clean) Is it really worth it in a performance-critical code as this? Maybe we could shuffle things around to HttpHeader and HttpHeaderEntry's ctor and dtor ..<br><br></div>More to come as I cross them.<br></div>(work in progress at lp:~kinkie/squid/coverity-fixes)<br clear="all"><div><div><div><div><div><div><div><div><div><br>-- <br><div class="gmail_signature">    Francesco</div>
</div></div></div></div></div></div></div></div></div></div>