<div dir="ltr"><div><div>Hi,<br></div>   In this case it's not needed and it clashes with the (well-reasoned) idea in the code not to have a default clause in the swtich statement at FtpServer.cc:559 so that compilers can check that all cases are covered.<br></div><div>Thanks!<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 20, 2015 at 6:03 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/20/2015 06:10 AM, Kinkie wrote:<br>
<br>
>   the attached patch turns FtpServer::EarlyErrorKind from a C enum into<br>
> c++11 strongly-typed enum. The code did not require any change, just a<br>
> bit of search-and-replace.<br>
<br>
</span>Consider adding begin/end markers (for your EnumIteration code) while<br>
you rewriting/renaming this (or any other) enum.<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>