From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11701 invoked by alias); 7 Nov 2012 20:11:19 -0000 Received: (qmail 11692 invoked by uid 22791); 7 Nov 2012 20:11:17 -0000 X-SWARE-Spam-Status: No, hits=-6.9 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_SPAMHAUS_DROP,RCVD_IN_DNSWL_HI,RCVD_IN_HOSTKARMA_W,RP_MATCHES_RCVD,SPF_HELO_PASS,TW_CN X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 07 Nov 2012 20:11:09 +0000 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id qA7KB9sH006717 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 7 Nov 2012 15:11:09 -0500 Received: from brno.lan (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id qA7KB83l018405 for ; Wed, 7 Nov 2012 15:11:08 -0500 Subject: [PATCH] Always include defs.h first. To: gdb-patches@sourceware.org From: Pedro Alves Date: Wed, 07 Nov 2012 20:11:00 -0000 Message-ID: <20121107201107.25258.47267.stgit@brno.lan> User-Agent: StGit/0.16 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit 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: 2012-11/txt/msg00166.txt.bz2 defs.h should always be the first included header in a .c file. In turn, this means that a foo.h header should not need to include defs.h, as the .c file always includes defs.h before including foo.h. I did $ grep "\"defs\.h" *.c -B 3 | grep include | grep -v "\"defs" and $ grep "\"defs\.h" *.h and hopefully caught all cases. Applied. 2012-11-07 Pedro Alves * arm-tdep.c: Make defs.h be the first include. * coff-pe-read.c: Ditto. * gnu-nat.c: Ditto. * go32-nat.c: Ditto. * i386-nat.c: Ditto. * ppcnbsd-nat.c: Ditto. * ada-varobj.h: Don't include defs.h. * i386-darwin-tdep.h: Ditto. * i386-nat.h: Ditto. --- gdb/ada-varobj.h | 1 - gdb/arm-tdep.c | 3 ++- gdb/coff-pe-read.c | 3 ++- gdb/gnu-nat.c | 3 ++- gdb/go32-nat.c | 3 ++- gdb/i386-darwin-tdep.h | 1 - gdb/i386-nat.c | 2 +- gdb/i386-nat.h | 2 -- gdb/ppcnbsd-nat.c | 3 ++- 9 files changed, 11 insertions(+), 10 deletions(-) diff --git a/gdb/ada-varobj.h b/gdb/ada-varobj.h index 2ef1a70..21f4cec 100644 --- a/gdb/ada-varobj.h +++ b/gdb/ada-varobj.h @@ -20,7 +20,6 @@ #ifndef ADA_VAROBJ_H #define ADA_VAROBJ_H -#include "defs.h" #include "varobj.h" struct value; diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index 05a030a..fbc2479 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -18,9 +18,10 @@ You should have received a copy of the GNU General Public License along with this program. If not, see . */ +#include "defs.h" + #include /* XXX for isupper (). */ -#include "defs.h" #include "frame.h" #include "inferior.h" #include "gdbcmd.h" diff --git a/gdb/coff-pe-read.c b/gdb/coff-pe-read.c index 66c7c82..fd55170 100644 --- a/gdb/coff-pe-read.c +++ b/gdb/coff-pe-read.c @@ -21,9 +21,10 @@ Contributed by Raoul M. Gough (RaoulGough@yahoo.co.uk). */ +#include "defs.h" + #include "coff-pe-read.h" -#include "defs.h" #include "bfd.h" #include "gdbtypes.h" diff --git a/gdb/gnu-nat.c b/gdb/gnu-nat.c index 2ca01e1..b34935e 100644 --- a/gdb/gnu-nat.c +++ b/gdb/gnu-nat.c @@ -22,6 +22,8 @@ along with this program. If not, see . */ +#include "defs.h" + #include #include #include @@ -52,7 +54,6 @@ #include -#include "defs.h" #include "inferior.h" #include "symtab.h" #include "value.h" diff --git a/gdb/go32-nat.c b/gdb/go32-nat.c index d848017..ef1da91 100644 --- a/gdb/go32-nat.c +++ b/gdb/go32-nat.c @@ -82,9 +82,10 @@ GDB does not use those as of this writing, and will never need to. */ +#include "defs.h" + #include -#include "defs.h" #include "i386-nat.h" #include "inferior.h" #include "gdbthread.h" diff --git a/gdb/i386-darwin-tdep.h b/gdb/i386-darwin-tdep.h index 8be9946..4732b2d 100644 --- a/gdb/i386-darwin-tdep.h +++ b/gdb/i386-darwin-tdep.h @@ -19,7 +19,6 @@ #ifndef __I386_DARWIN_TDEP_H__ #define __I386_DARWIN_TDEP_H__ -#include "defs.h" #include "frame.h" /* Mapping between the general-purpose registers in Darwin x86 thread_state diff --git a/gdb/i386-nat.c b/gdb/i386-nat.c index e7d9b4d..559bc29 100644 --- a/gdb/i386-nat.c +++ b/gdb/i386-nat.c @@ -18,8 +18,8 @@ You should have received a copy of the GNU General Public License along with this program. If not, see . */ -#include "i386-nat.h" #include "defs.h" +#include "i386-nat.h" #include "breakpoint.h" #include "command.h" #include "gdbcmd.h" diff --git a/gdb/i386-nat.h b/gdb/i386-nat.h index c06993c..be563ef 100644 --- a/gdb/i386-nat.h +++ b/gdb/i386-nat.h @@ -20,8 +20,6 @@ You should have received a copy of the GNU General Public License along with this program. If not, see . */ -#include "defs.h" - #ifndef I386_NAT_H #define I386_NAT_H 1 diff --git a/gdb/ppcnbsd-nat.c b/gdb/ppcnbsd-nat.c index 8f73283..2b7837d 100644 --- a/gdb/ppcnbsd-nat.c +++ b/gdb/ppcnbsd-nat.c @@ -19,13 +19,14 @@ You should have received a copy of the GNU General Public License along with this program. If not, see . */ +#include "defs.h" + #include #include #include #include #include -#include "defs.h" #include "gdbcore.h" #include "inferior.h" #include "regcache.h"