<div dir="ltr"><div>Hi,<br></div>  as we dropped icc support (which doesn't support override) I've reverted the change which is now useless. We will reassess the situation when icc will fix the other issues we found today.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 26, 2015 at 5:13 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/26/2015 06:05 AM, Kinkie wrote:<br>
> Hi,<br>
>      the attached patch sets up a new autoconf test to detect whether<br>
> the compiler supports c++11's "override" keyword, and defines a<br>
> generally-available "OVERRIDE" preprocessor macro which expands to<br>
> "override" if support is present and nothing if it isn't.<br>
><br>
> This patch is necessary to support icc 13.1.3 which claims to be c++11<br>
> but doesn't support this keyword.<br>
<br>
<br>
</span>I do not see why we should pollute our code with OVERRIDE screams. If we<br>
require C++11 support, we should not #define override or any other<br>
standard and commonly used keywords. If somebody wants to compile with<br>
icc 13.1.3, give them a command to remove override keywords from sources.<br>
<br>
If you disagree, please use low-case letters for that #define. For<br>
example, squid_override.<br>
<br>
<br>
Thank you,<br>
<br>
Alex.<br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">    Francesco</div>
</div>