php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #46237 Repeated compiler warnings (ZE + PHP)
Submitted: 2008-10-05 13:18 UTC Modified: 2009-05-04 15:24 UTC
From: kalle@php.net Assigned: pajoye (profile)
Status: Closed Package: Feature/Change Request
PHP Version: 5.3.0alpha2 OS: Windows Vista
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
49 + 6 = ?
Subscribe to this entry?

 
 [2008-10-05 13:18 UTC] kalle@php.net
Description:
------------
I've noticed that theres really many repeated compiler warnings especially on Windows, so here goes:


Zend Engine:
HEAD: http://phpfi.com/360836
PHP_5_3: http://phpfi.com/360837

PHP:
In both HEAD and PHP_5_3 theres a repeated compiler warning C4142 (benign redefinition of type):
C:\Program Files\Microsoft SDKs\Windows\v6.0A\include\Ws2tcpip.h(502) : warning C4142: benign redefinition of type

I looked at Ws2tcpip.h, and it seems to be redefining socklen_t which is in the Windows SDK, its only defined if _WIN32_WINNT is >= 6, so it must be included before the generated config file (main/config.w32.h) where _WIN32_WINNT is defined to 5. Also main/php.h seems to be defining socklen_t ifndef HAVE_SOCKLEN_T, which im unsure if the Windows SDK has.


Note that these are not critical for actual running PHP, but it would be great to have a cleaner screen on compile ;)


Resources (MSDN on compiler warnings):
C4090: http://msdn.microsoft.com/en-us/library/k77bkb8d.aspx
C4101: http://msdn.microsoft.com/en-us/library/c733d5h9.aspx
C4142: http://msdn.microsoft.com/en-us/library/cbs8z6wh.aspx


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2009-05-04 15:24 UTC] pajoye@php.net
This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.


 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sun Apr 14 17:01:28 2024 UTC