It appears that we should be able to include strings.h only, except msvc does not...

and at least some linux implementations do not work unless you include both, which seems to be against spec, but should not hurt.  The most correct thing to do here would be to include strings.h then string.h wrapped in autoconf detected defines, but this should work for now.  If it is still an issue going forward, we will need to add some defines for detected presence of some header files to the public header files, which should be done with care to avoid define collisions in public header files.

http://www.opengroup.org/onlinepubs/009695399/basedefs/strings.h.html
http://www.opengroup.org/onlinepubs/009695399/basedefs/string.h.html

git-svn-id: http://svn.freeswitch.org/svn/freeswitch/trunk@4895 d0543943-73ff-0310-b7d9-9358b9ac24b2

diff --git a/src/include/switch.h b/src/include/switch.h
index f0f91c90e417ca6a2134bb86fd2f717f1f372124..0f00b6ec8f1b573f1927af4bf7b7f266af459009 100644 (file)
--- a/src/include/switch.h
+++ b/src/include/switch.h
@@ -59,11 +59,10 @@
 #include <setjmp.h>
 #include <ctype.h>
 #include <fcntl.h>
-#ifdef _MSC_VER
-#include <string.h>
-#else
+#ifndef _MSC_VER
 #include <strings.h>
 #endif
+#include <string.h>
 #include <sys/stat.h>
 #include <limits.h>
 #include <sys/types.h>