Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_3) / zend_API.c zend_builtin_functions.c zend_compile.h zend_execute.c zend_execute.h zend_execute_API.c zend_globals.h zend_vm_def.h zend_vm_execute.h zend_vm_execute.skl zend_vm_gen.php /tests

I tried MSVC5 too. no warnings.

 > cl

Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 12.00.8804 for 80x86
Copyright (C) Microsoft Corp 1984-1998. All rights reserved.

Dmitry.

Steph Fox wrote:
> MSVC6 - which is still used for official php.net builds + snaps because 
> everything Apache not CGI/CLI fails otherwise.
> 
> Configure line is: --with-bz2 --enable-phar --enable-zip=shared. I don't 
> recommend trying that @ present (phar build's broken for me) but I think 
> you'll agree it doesn't affect ZE warnings.
> 
> - Steph
> 
> 
>> Hi Steph,
>>
>> What compiler/configure option do you use?
>> I don't have such warnings.
>>
>> Thanks. Dmitry.
>>
>> Steph Fox wrote:
>>> Hi Dmitry,
>>>
>>> This change led to an interesting make output here. The whole thing 
>>> looks like this throughout:
>>>
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>> sanitizing_filters.c
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>> ftp.c
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>> php_ftp.c
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>> hash.c
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>> hash_adler32.c
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>> hash_crc32.c
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>> hash_gost.c
>>> Zend\zend_execute.h(229) : warning C4018: '>' : signed/unsigned mismatch
>>>
>>> I wasn't aware of zend_execute.h being called so many times until 
>>> now, but maybe that's just because there's a warning now that wasn't 
>>> there before? I can kill the warning easily enough:
>>>
>>> ZEND_VM_STACK_GROW_IF_NEEDED((int)size);
>>>
>>> but I've no idea if that breaks anything delicate.
>>>
>>> - Steph
>>>
>>>
>>> ----- Original Message ----- From: "Dmitry Stogov" <[email protected]>
>>> To: <[email protected]>
>>> Sent: Thursday, January 24, 2008 9:41 AM
>>> Subject: [ZEND-ENGINE-CVS] cvs: ZendEngine2(PHP_5_3) / zend_API.c 
>>> zend_builtin_functions.c zend_compile.h zend_execute.c zend_execute.h 
>>> zend_execute_API.c zend_globals.h zend_vm_def.h zend_vm_execute.h 
>>> zend_vm_execute.skl zend_vm_gen.php /tests bug41209.phpt php-src NEWS
>>>
>>>
>>>> dmitry Thu Jan 24 09:41:41 2008 UTC
>>>>
>>>>  Modified files:              (Branch: PHP_5_3)
>>>>    /php-src NEWS
>>>>    /ZendEngine2 zend_API.c zend_builtin_functions.c zend_compile.h
>>>>                zend_execute.c zend_execute.h zend_execute_API.c
>>>>                zend_globals.h zend_vm_def.h zend_vm_execute.h
>>>>                zend_vm_execute.skl zend_vm_gen.php
>>>>    /ZendEngine2/tests bug41209.phpt
>>>>  Log:
>>>>  Changed EG(argument_stack) implementation.
>>>>
>>>>
>>>
>>>
>>> -------------------------------------------------------------------------------- 
>>>
>>>
>>>
>>>
>>>> -- 
>>>> Zend Engine CVS Mailing List (http://cvs.php.net/)
>>>> To unsubscribe, visit: http://www.php.net/unsub.php
>>>
>