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

Julien Fischer jfischer at opturion.com
Wed Apr 2 10:51:54 AEDT 2014


On Wed, Apr 2, 2014 at 10:04 AM, Paul Bone <paul at bone.id.au> wrote:

> 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?
>

We compile with -Wno-unused (see the mgnuc script).
(If we didn't compile with -Wno-unused we would get large numbers of
warnings
about the generated code.)

Cheers,
Julien.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20140402/130b5845/attachment.html>


More information about the reviews mailing list