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

Oh. I meant MSVC6-SP5.
Could you please verify the version with "cl" command.

Dmitry.

Steph Fox wrote:
> But not MSVC6?
> 
> Why would I get a compiler warning and you not? That doesn't make a lot 
> of sense.
> 
> ----- Original Message ----- From: "Dmitry Stogov" <[email protected]>
> To: "Steph Fox" <[email protected]>
> Cc: "Dmitry Stogov" <[email protected]>; <[email protected]>
> Sent: Thursday, January 24, 2008 4:18 PM
> Subject: 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
>>>>>
>>>
> 
>