#45698 [Asn]: Doesn't compile with VC6 - Include problem

ID: 45698 Updated by: pajoye@php.net Reported By: Keisial at gmail dot com Status: Assigned Bug Type: Compile Failure Operating System: Windows - VC6 PHP Version: 5.3CVS-2008-08-03 (snap) -Assigned To: pajoye +Assigned To: stas New Comment: Stas, can you commit the patch please? Previous Comments: ------------------------------------------------------------------------ [2008-08-03 12:27:13] derick@php.net This will probably fix the compile failure that Steph and I were getting... ------------------------------------------------------------------------ [2008-08-03 12:23:45] Keisial at gmail dot com Description: ------------ 5.3 doesn't compile with VC6. When including winsock2.h at zend_config.w32.h it includes windows.h which in turn includes winsock stuff before a SOCKET is defined. Compilation fails with error on mswsock.h (This doesn't happen including only those two headers, so some defines are modifying the defualt behaviour) Changing the order to include windows.h before winsock2.h fixes it: --- zend_config.w32.h 2008-07-17 20:33:46.000000000 +0200 +++ zend_config.w32.h 2008-08-03 14:19:56.593750000 +0200 @@ -35,8 +35,8 @@ #ifndef ZEND_INCLUDE_FULL_WINDOWS_HEADERS #define WIN32_LEAN_AND_MEAN #endif -#include <winsock2.h> #include <windows.h> +#include <winsock2.h> #include <float.h> ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=45698&edit=1