From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21067 invoked by alias); 30 Aug 2010 14:08:28 -0000 Received: (qmail 21055 invoked by uid 22791); 30 Aug 2010 14:08:27 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 30 Aug 2010 14:08:22 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 815DA2BAC17; Mon, 30 Aug 2010 10:08:20 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id CxQ07LsYexFe; Mon, 30 Aug 2010 10:08:20 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 2BFDE2BAC00; Mon, 30 Aug 2010 10:08:20 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id AC088F599F; Mon, 30 Aug 2010 16:08:14 +0200 (CEST) Date: Mon, 30 Aug 2010 14:08:00 -0000 From: Joel Brobecker To: Mark Kettenis Cc: eliz@gnu.org, jan.kratochvil@redhat.com, gdb-patches@sourceware.org Subject: Re: [patch 1/9]#2 Rename `enum target_signal' to target_signal_t Message-ID: <20100830140814.GE2986@adacore.com> References: <20100830071040.GB6831@host1.dyn.jankratochvil.net> <201008300814.o7U8ECNZ009010@glazunov.sibelius.xs4all.nl> <20100830082407.GA6881@host1.dyn.jankratochvil.net> <201008301111.o7UBB0c8030327@glazunov.sibelius.xs4all.nl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201008301111.o7UBB0c8030327@glazunov.sibelius.xs4all.nl> User-Agent: Mutt/1.5.20 (2009-06-14) Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2010-08/txt/msg00512.txt.bz2 > 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