Re: [INTERNALS-WIN] #45456 (_WIN32_WINNT) and fix VC9, zend/

[email protected] ("Pierre Joye")
Newsgroups php.internals.win
Message-ID <[email protected]>
hi!

On Mon, Jul 14, 2008 at 9:23 PM, Stanislav Malyshev <[email protected]> wrote:

>> Is (or can be) zend_config.w32.h included when needed then? Then only
>> this file will include main/config.w32.h.
>
> It should be. If it isn't included somewhere, no problem to add it. But
> zend.h includes it, and zend_alloc.c includes zend.h - so it should already
> happen.
> BTW, same story should happen on TSRM side.
>
>> About having a specific zend_config.w32.h to replace config.w32.h: I'm
>> not sure it is worth the effort (for windows). We can always do it
>
> It already exists, that's how it worked since time immemorial, and how it
> works now, so it's zero work :)

Oh right, I will update the patch to simply drop the  _WIN32_WINNT and
zend_config.w32.h if necessary. I will also add it to
tsrm_config.w32.h.

-- 
Pierre

http://blog.thepimp.net | http://www.libgd.org
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.