<div dir="ltr">Hi,<div>   this patch:</div><div>- removes unused wordlistDup</div><div>- implements proper constructors for wordlist, making it MEMPROXY_CLASS-safe</div><div>- makes its destructor private to force clients to use wordlistDestroy instead</div><div>- implements wordlistChopHead to support the one case in the whole codebase where a wordlist element was legitimately deleted.</div><div><br></div><div>The code looks odd, but this is an unfortunate consequence of asymmetric expectations on the construct/destruct side: construct side requrires C++ semantics for MEMPROXY_CLASS, destruct side callers expect C semantics.</div><div><br></div><div>The code has been build- and run- tested.</div><div><br></div><div>Patch with big context attached.</div><div><br></div><div><div><br></div>-- <br><div class="gmail_signature">    Francesco</div>
</div></div>