Include process.h for _getpid on windows.
authorRalph Giles <giles@mozilla.com>
Tue, 23 Oct 2012 20:04:36 +0000 (13:04 -0700)
committerRalph Giles <giles@mozilla.com>
Tue, 23 Oct 2012 20:04:36 +0000 (13:04 -0700)
Previously the test programmes happened to link because _getpid
returns an int, which is the default prototype. Instead, include
the appropriate header to supply a real declaration, fixing the
associated warning.

I moved the getpid->_getpid define to the site of the header
inclusion to make clear what's going on, rather than have it
floating in win32/config.h.

tests/test_opus_decode.c
tests/test_opus_encode.c
win32/config.h

index 2c6a872..868869b 100644 (file)
@@ -38,6 +38,9 @@
 #include <time.h>
 #if (!defined WIN32 && !defined _WIN32) || defined(__MINGW32__)
 #include <unistd.h>
+#else
+#include <process.h>
+#define getpid _getpid
 #endif
 #include "opus.h"
 #include "test_opus_common.h"
index 01534fa..a9a1c58 100644 (file)
@@ -38,6 +38,9 @@
 #include <time.h>
 #if (!defined WIN32 && !defined _WIN32) || defined(__MINGW32__)
 #include <unistd.h>
+#else
+#include <process.h>
+#define getpid _getpid
 #endif
 #include "opus_multistream.h"
 #include "opus.h"
index 6d32620..d46d0d5 100644 (file)
@@ -4,7 +4,6 @@
 #define CELT_BUILD            1
 
 #define inline __inline
-#define getpid _getpid
 
 #define USE_ALLOCA            1