From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18210 invoked by alias); 15 May 2006 17:12:10 -0000 Received: (qmail 18202 invoked by uid 22791); 15 May 2006 17:12:09 -0000 X-Spam-Check-By: sourceware.org Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.31.1) with ESMTP; Mon, 15 May 2006 17:12:05 +0000 Received: from drow by nevyn.them.org with local (Exim 4.54) id 1Ffgbx-0000GJ-OD for gdb-patches@sourceware.org; Mon, 15 May 2006 13:12:01 -0400 Date: Mon, 15 May 2006 17:16:00 -0000 From: Daniel Jacobowitz To: gdb-patches@sourceware.org Subject: Re: [rfc] gdb_stdint.h. Message-ID: <20060515171201.GD385@nevyn.them.org> Mail-Followup-To: gdb-patches@sourceware.org References: <20060505172713.GF31029@nevyn.them.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060505172713.GF31029@nevyn.them.org> User-Agent: Mutt/1.5.11+cvs20060403 X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-05/txt/msg00340.txt.bz2 On Fri, May 05, 2006 at 01:27:13PM -0400, Daniel Jacobowitz wrote: > You want uintptr_t? I got your uintptr_t right here. > > This patch creates a header in the build directory named gdb_stdint.h. > It defines all of the things which a stdint.h ought to define. It uses > the system's stdint.h if there is one, and provides anything missing, > up to and including the whole file. > > There is, of course, a huge hairy chunk of m4 and portability knowledge > that you don't see in this patch. That's because it's already in > config/ and was written for GCC. I'm OK with that :-) It means it's > quite well tested at this point. > > This would be enough to allow us to start using uint64_t, uintptr_t, > et cetera in the common code of GDB. Which, in my opinion, would be a > good idea. They're nice to have around. > > Any comments? > 2006-05-05 Daniel Jacobowitz > > * configure.ac: Use GCC_HEADER_STDINT. > * acinclude.m4: Include stdint.m4. > * Makefile.in (gdb_stdint_h): Define. > (distclean): Remove gdb_stdint.h. > (Makefile, stamp-h): Update rules to generate only the correct > files. > (gdb_stdint.h, stamp-int): New rules. > * config.in, configure: Regenerated. Comments sounded supportive to me! I have committed it. Debian unstable appears to have gone up to autoconf 2.59d, so configure has changed a bit with this commit; I think there won't be any problems that affect GDB, but please do let me know if I'm wrong. -- Daniel Jacobowitz CodeSourcery