diff options
| author | Tom Tromey <tromey@redhat.com> | 2013-11-06 07:36:46 -0700 |
|---|---|---|
| committer | Tom Tromey <tromey@redhat.com> | 2013-11-18 13:29:01 -0700 |
| commit | 161d1bec16764585e987c8d804df0fb560a0d491 (patch) | |
| tree | 57fe668db971f2368399a241b9e4effec80b7167 | |
| parent | 2978b11100a6318e05b8bc2d7923d9f735ef5c6e (diff) | |
| download | gdb-161d1bec16764585e987c8d804df0fb560a0d491.zip gdb-161d1bec16764585e987c8d804df0fb560a0d491.tar.gz gdb-161d1bec16764585e987c8d804df0fb560a0d491.tar.bz2 | |
don't check for stddef.h
gdb already unconditionally includes stddef.h in many places.
I think there is no reason to check for its existence.
Also, Zack Weinberg's header file survey agrees:
http://hacks.owlfolio.org/header-survey/
This patch removes the configure check and the inclusion guards.
It also removes a redundant inclusion that I noticed in defs.h.
2013-11-18 Tom Tromey <tromey@redhat.com>
* config.in: Rebuild.
* configure: Rebuild.
* configure.ac: Don't check for stddef.h.
* defs.h: Unconditionally include stddef.h. Remove duplicate
inclusion.
| -rw-r--r-- | gdb/ChangeLog | 8 | ||||
| -rw-r--r-- | gdb/config.in | 3 | ||||
| -rwxr-xr-x | gdb/configure | 2 | ||||
| -rw-r--r-- | gdb/configure.ac | 2 | ||||
| -rw-r--r-- | gdb/defs.h | 6 |
5 files changed, 10 insertions, 11 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index fc6336c..3a6f452 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,13 @@ 2013-11-18 Tom Tromey <tromey@redhat.com> + * config.in: Rebuild. + * configure: Rebuild. + * configure.ac: Don't check for stddef.h. + * defs.h: Unconditionally include stddef.h. Remove duplicate + inclusion. + +2013-11-18 Tom Tromey <tromey@redhat.com> + * common/common.m4 (GDB_AC_COMMON): Don't use AC_HEADER_DIRENT. * common/gdb_dirent.h: Remove. * common/filestuff.c: Use dirent.h. diff --git a/gdb/config.in b/gdb/config.in index 4e3f6de..802127f 100644 --- a/gdb/config.in +++ b/gdb/config.in @@ -378,9 +378,6 @@ /* Define to 1 if the system has the type `socklen_t'. */ #undef HAVE_SOCKLEN_T -/* Define to 1 if you have the <stddef.h> header file. */ -#undef HAVE_STDDEF_H - /* Define to 1 if you have the <stdint.h> header file. */ #undef HAVE_STDINT_H diff --git a/gdb/configure b/gdb/configure index b1a4d48..1d1a486 100755 --- a/gdb/configure +++ b/gdb/configure @@ -8756,7 +8756,7 @@ fi # elf_hp.h is for HP/UX 64-bit shared library support. for ac_header in nlist.h machine/reg.h poll.h sys/poll.h proc_service.h \ - thread_db.h stddef.h \ + thread_db.h \ stdlib.h sys/fault.h \ sys/file.h sys/filio.h sys/ioctl.h sys/param.h \ sys/resource.h sys/procfs.h sys/ptrace.h ptrace.h \ diff --git a/gdb/configure.ac b/gdb/configure.ac index 6025e89..d6b75d6 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -1075,7 +1075,7 @@ fi AC_HEADER_STDC # elf_hp.h is for HP/UX 64-bit shared library support. AC_CHECK_HEADERS([nlist.h machine/reg.h poll.h sys/poll.h proc_service.h \ - thread_db.h stddef.h \ + thread_db.h \ stdlib.h sys/fault.h \ sys/file.h sys/filio.h sys/ioctl.h sys/param.h \ sys/resource.h sys/procfs.h sys/ptrace.h ptrace.h \ @@ -48,9 +48,7 @@ included, so it's ok to blank out gstdint.h. */ #define GCC_GENERATED_STDINT_H 1 -#ifdef HAVE_STDDEF_H #include <stddef.h> -#endif #include <unistd.h> @@ -606,10 +604,6 @@ enum gdb_osabi /* From other system libraries */ -#ifdef HAVE_STDDEF_H -#include <stddef.h> -#endif - #ifdef HAVE_STDLIB_H #include <stdlib.h> #endif |