[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