[m-rev.] diff: delete unused local variables in trace library

Paul Bone paul at bone.id.au
Wed Apr 2 10:04:09 AEDT 2014


On Thu, Mar 20, 2014 at 11:47:50AM +1100, Julien Fischer wrote:
>
> Branches: master, 14.01
>
> When we originally added support for clang, most of its warnings were
> disabled (and still are); I've been trying it out with more warning  
> options enabled and this is the first (of probably a few) changes that 
> fix
> up things that it is complaining about.
>
> --------------------
>
> Delete unused local variables in trace library.

That seems weird to me.  Shouldn't gcc also report this as a warning?

There's nothing wrong with your change, I'm just curious why we're not
getting this warning with any C compiler.

Thanks.


-- 
Paul Bone



More information about the reviews mailing list