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: Mark Kettenis <mark dot kettenis at xs4all dot nl>
- To: jan dot kratochvil at redhat dot com
- Cc: gdb-patches at sourceware dot org
- Date: Mon, 30 Aug 2010 10:14:12 +0200 (CEST)
- Subject: Re: [patch 1/9]#2 Rename `enum target_signal' to target_signal_t
- References: <20100830071040.GB6831@host1.dyn.jankratochvil.net>
> Date: Mon, 30 Aug 2010 09:10:40 +0200
> From: Jan Kratochvil <jan.kratochvil@redhat.com>
>
> series #2 of: http://sourceware.org/ml/gdb-patches/2010-07/msg00408.html
>
> Hi,
>
> there is no compiled code change in this patch. This patch just renames `enum
> target_signal' to target_signal_t (typedef) so the code is more flexible for
> the later changes. No other change is made.
The _t suffix is actually reserved by POSIX, so using it is actually a
bad idea. Perhaps change it to gdb_target_signal, or just
target_signal (making it a typedef for an anonymous enum)?