<div dir="ltr"><div>Hi,<br></div>  the attached patch implements this suggestion renaming to SBufHashCommon_ (it seems to be a common pattern) and adding a comment.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 24, 2015 at 8:51 PM, Alex Rousskov <span dir="ltr"><<a href="mailto:rousskov@measurement-factory.com" target="_blank">rousskov@measurement-factory.com</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 08/24/2015 11:13 AM, Kinkie wrote:<br>
>   this is part 2 of the coverity-fixes merge: SBuf hash.<br>
> This code allows to have std::unordered_maps where the key is a SBuf, in<br>
> case-sensitive and case-insensitive variants. The hashing is the same as<br>
> used in the current Squid hash implementation (from lib/hash.cc) in<br>
> order not to upset the current uses. More specific algorithms can be<br>
> implemented by future users if they wish to.<br>
<br>
<br>
</span>> +lowSBufHash (const SBuf & sbuf, bool caseInsensitive = false) noexcept<br>
<br>
I recommend using trailing underscore to mark low-level, "private" code<br>
instead of "low" which may be confused with lower-case-related method,<br>
especially in an SBuf context. Not important.<br>
<span class="HOEnZb"><font color="#888888"><br>
Alex.<br>
<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">    Francesco</div>
</div>