Wireshark mailing list archives

Re: New GCC, new option required?


From: Bill Meier <wmeier () newsguy com>
Date: Wed, 13 Jul 2011 10:23:48 -0400

On 7/13/2011 8:05 AM, Jaap Keuter wrote:
Hi list,

I think we need a new GCC option added to configure.in:

AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wno-unused-but-set-variable)

This option came into existence with GCC 4.6 and is causing havoc all over.
I've been going through a lot of dissectors, many of which have either:
* unused set variables
* variables set for a legitimate purpose, but not used (yet)
* bugs


I've fixes for a number of the (non-generated) dissectors and will commit them today.

Re:

> AC_WIRESHARK_GCC_CFLAGS_CHECK(-Wno-unused-but-set-variable)

Is there a (reasionably easy) way to use this option only when compiling in the .../dissectors directory ?

(I think that I've previously addressed the "unused_but_set" warnings for many (all ?) of the other directories). (SVN 37600 & following...)

Bill

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:wireshark-dev-request () wireshark org?subject=unsubscribe


Current thread: