<div dir="ltr">Thanks, done in r200479.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 30 January 2014 18:49, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.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="HOEnZb"><div class="h5"><br>
On 30/01/2014 18:42, Renato Golin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Modern compilers (Clang 3.4, GCC 4.8) warn on variadic macros being<br>
introduced in C99, which produces a huge number of useless diagnostics<br>
since this macro is unused in the whole project.<br>
<br>
All tests compile and pass.<br>
</blockquote>
<br></div></div>
In the comment just go for "// Disabled to silence LLVM build warnings." to point out it's a local change. The versions are fuzzy anyway so not much value in listing them.<br>
<br>
Also can you disable the code with #if 0 instead of a comment?<br>
<br>
Otherwise looks fine as a way of silencing the noise, thanks<span class="HOEnZb"><font color="#888888"><br>
<br>
Alp.<br>
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</font></span></blockquote></div><br></div>