<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 23, 2014 at 2:58 AM, Patrik Hägglund H <span dir="ltr"><<a href="mailto:patrik.h.hagglund@ericsson.com" target="_blank">patrik.h.hagglund@ericsson.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">> In all seriousness, we need to find a solution to this soon, or revert this change.<span> 
</span><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
</div><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">+1<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">> For those who want to use old/pre-installed/official release clangs rather than more recent iterations, they can just
 disable –<span>Werror</span> (it's not the default anyway).<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">For us, disabling –<span>Werror</span> is currently not an option. If this is the intention, I think the
 policy should be updated first. (I would rather like the policy to be strengthened, to support –<span>Werror</span> builds with the latest official release of _<i>both</i>_ clang and
<span>gcc</span>.)</span></p></blockquote></div><br>While I haven't really had time, and I didn't even write the original patch, I've already described exactly how to fix this by disabling the warning with a version check in CMake... It should be really straightforward to implement. I guess if for some reason none have done it I will, but I don't even have a version of Clang to test it with, so I'm somewhat inclined for someone who does to write this patch...</div>
</div>