Re: [rfc] gdb_stdint.h.

Daniel Jacobowitz
2006-05-15T17:12:10Z

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