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: [rfc] gdb_stdint.h.


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  <dan@codesourcery.com>
> 
> 	* 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


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