This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA 6/6] Add -Wunused-but-set-* to build
- From: Trevor Saunders <tbsaunde at tbsaunde dot org>
- To: Tom Tromey <tom at tromey dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 7 Jun 2016 22:37:51 -0400
- Subject: Re: [RFA 6/6] Add -Wunused-but-set-* to build
- Authentication-results: sourceware.org; auth=none
- References: <1465248812-23902-1-git-send-email-tom at tromey dot com> <1465248812-23902-7-git-send-email-tom at tromey dot com>
On Mon, Jun 06, 2016 at 03:33:32PM -0600, Tom Tromey wrote:
> This adds -Wunused-but-set-variable and -Wunused-but-set-parameter to
> configure.
great, I started on that a little while ago but didn't get to pushing
these bits upstream.
> +++ b/gdb/warning.m4
> @@ -39,7 +39,7 @@ fi
> build_warnings="-Wall -Wpointer-arith \
> -Wno-unused -Wunused-value -Wunused-function \
> -Wno-switch -Wno-char-subscripts \
> --Wempty-body"
> +-Wempty-body -Wunused-but-set-parameter -Wunused-but-set-variable"
isn't everything in -Wunused enabled now? can we just delete -Wno-unused
and use -Wall to get us -Wunused?
Trev