From: Paul Eggert <eggert@cs.ucla.edu>
Date: Fri, 20 Nov 2015 08:40:45 +0000 (-0800)
Subject: Include-file tweaks for modules
X-Git-Tag: emacs-25.0.90~721
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=c69cec404cbc6b1570c42ca03e0cb42b410d7690;p=emacs.git

Include-file tweaks for modules

* src/dynlib.c, src/emacs-module.c: Include <config.h> first.
* src/dynlib.h: Do not include config.h.
It’s every .c file’s responsibility to include config.h first.
* src/emacs-module.c: Include emacs-module.h immediately after
config.h, to test that emacs-module.h doesn’t depend on
include files other than config.h.
---

diff --git a/src/dynlib.c b/src/dynlib.c
index ed4f157d0fe..1b66c4ad8e8 100644
--- a/src/dynlib.c
+++ b/src/dynlib.c
@@ -24,6 +24,8 @@ along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.  */
    If you think the abstraction is too leaky use libltdl (libtool),
    don't reinvent the wheel by fixing this one.  */
 
+#include <config.h>
+
 #include "dynlib.h"
 
 #if defined _WIN32
diff --git a/src/dynlib.h b/src/dynlib.h
index bd03666b860..1282c4fd719 100644
--- a/src/dynlib.h
+++ b/src/dynlib.h
@@ -20,7 +20,6 @@ along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.  */
 #ifndef DYNLIB_H
 #define DYNLIB_H
 
-#include <config.h>
 #include <stdbool.h>
 
 typedef void *dynlib_handle_ptr;
diff --git a/src/emacs-module.c b/src/emacs-module.c
index e885af5de8f..25c5e019881 100644
--- a/src/emacs-module.c
+++ b/src/emacs-module.c
@@ -17,14 +17,16 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.  */
 
+#include <config.h>
+
+#include "emacs-module.h"
+
 #include <stdbool.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <stdio.h>
 
-#include <config.h>
 #include "lisp.h"
-#include "emacs-module.h"
 #include "dynlib.h"
 #include "coding.h"
 #include "verify.h"