This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch 1/9]#2 Rename `enum target_signal' to target_signal_t
- From: Pedro Alves <pedro at codesourcery dot com>
- To: gdb-patches at sourceware dot org
- Cc: Daniel Jacobowitz <dan at codesourcery dot com>, Jan Kratochvil <jan dot kratochvil at redhat dot com>, Joel Brobecker <brobecker at adacore dot com>, Eli Zaretskii <eliz at gnu dot org>, Mark Kettenis <mark dot kettenis at xs4all dot nl>
- Date: Wed, 1 Sep 2010 21:08:56 +0100
- Subject: Re: [patch 1/9]#2 Rename `enum target_signal' to target_signal_t
- References: <E1Oq55N-0006ia-B0@fencepost.gnu.org> <201009012059.36696.pedro@codesourcery.com> <20100901200621.GA11085@caradoc.them.org>
On Wednesday 01 September 2010 21:06:24, Daniel Jacobowitz wrote:
> On Wed, Sep 01, 2010 at 08:59:36PM +0100, Pedro Alves wrote:
> > But, here's another idea of how to get compiler warnings/errors,
> > that I think is more transparent to code throughout:
> >
> > /* An empty struct. It's the instances we care about. */
> > struct gdb_signal_1
> > {
> > };
>
> Please don't do this! It makes things much harder to debug.
Not with a gdb_signal pretty printer. ;-)
--
Pedro Alves