Although this sounds an extremely valid change, it breaks binary so
I'm against on 5.3.
Also, introducing type hints doesn't means that also core functions
should follow it? Because currently '1' is converted to true.
So in microtime for example... it should not support
microtime('true'), but microtime(true) only.
This is a change in zend_parse_parameters that automagically converts
to correct type. That means more logic that if this is applied to
userland, the same should be applied to internal functions.
Just my 0.02.
Cheers,
On Wed, Jul 1, 2009 at 3:12 PM, Hannes
Magnusson<hannes.magnus...@gmail.com> wrote:
> On Wed, Jul 1, 2009 at 18:59, Ilia Alshanetsky<i...@prohost.org> wrote:
>> The patch is available here: http://ia.gd/patch/type_hint_53.txt
>
> It is missing minor build fix for ext/reflection, see
> http://pastebin.com/f50db9aa1
>
> Other then that, I'm definitely +1 on this
>
> -Hannes
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
--
Guilherme Blanco - Web Developer
CBC - Certified Bindows Consultant
Cell Phone: +55 (16) 9215-8480
MSN: guilhermebla...@hotmail.com
URL: http://blog.bisna.com
São Paulo - SP/Brazil
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
