From 9cd661e746708a76761ff1fda361c6276ee71ab1 Mon Sep 17 00:00:00 2001 From: Andrew Wedgbury Date: Mon, 7 Apr 2014 12:40:35 +0100 Subject: Make sure config.h is included before any system headers There was an issue recently in screen-share.c where config.h was not being included, resulting in the wrong definition for off_t being used on 32 bit systems. I checked and I don't think this problem is happening elsewhere, but to help avoid this sort of problem in the future, I went through and made sure that config.h is included first whenever system headers are included. The config.h header should be included before any system headers, failing to do this can result in the wrong type sizes being defined on certain systems, e.g. off_t from sys/types.h Signed-off-by: Andrew Wedgbury --- shared/os-compatibility.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'shared') diff --git a/shared/os-compatibility.h b/shared/os-compatibility.h index 10dbb7dd..172bb7ed 100644 --- a/shared/os-compatibility.h +++ b/shared/os-compatibility.h @@ -23,10 +23,10 @@ #ifndef OS_COMPATIBILITY_H #define OS_COMPATIBILITY_H -#include - #include "config.h" +#include + #ifdef HAVE_EXECINFO_H #include #else -- cgit v1.2.1