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

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 
> 
>