<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 2, 2014 at 10:04 AM, Paul Bone <span dir="ltr"><<a href="mailto:paul@bone.id.au" target="_blank">paul@bone.id.au</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, Mar 20, 2014 at 11:47:50AM +1100, Julien Fischer wrote:<br>
><br>
> Branches: master, 14.01<br>
><br>
> When we originally added support for clang, most of its warnings were<br>
> disabled (and still are); I've been trying it out with more warning<br>
> options enabled and this is the first (of probably a few) changes that<br>
> fix<br>
> up things that it is complaining about.<br>
><br>
> --------------------<br>
><br>
> Delete unused local variables in trace library.<br>
<br>
</div>That seems weird to me. Shouldn't gcc also report this as a warning?<br></blockquote><div><br></div><div>We compile with -Wno-unused (see the mgnuc script).</div><div>(If we didn't compile with -Wno-unused we would get large numbers of warnings<br>
</div><div>about the generated code.)</div><div><br></div><div>Cheers,</div><div>Julien.</div></div></div></div>