This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch 1/9]#2 Rename `enum target_signal' to target_signal_t


> http://www.opengroup.org/onlinepubs/009695399/functions/xsh_chap02_02.html
> 
> The _t suffix is listed on the last row of the first table on that page.

It does not matter much to me if we decide to allow types named with
a _t suffix. But the gdb_ prefix also increases the chances of avoiding
name collisions with external declarations. Especially in this particular
case, I feel that target_signal is sufficiently generic that we might
want to be extra careful.

-- 
Joel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]