Re: svn: /php/php-src/trunk/ NEWS Zend/tests/array_type_hint_001.phpt Zend/tests/hint/param_type_hint_001.phpt
Zend/tests/hint/param_type_hint_002.phpt Zend/tests/hint/param_type_hint_003.phpt Zend/tests/hint/param_type_hint_004.phpt Zend/t
| From: | Pierre Joye | Date: | Sat, 22 May 2010 15:22:30 +0000 |
| Subject: | Re: svn: /php/php-src/trunk/ NEWS Zend/tests/array_type_hint_001.phpt Zend/tests/hint/param_type_hint_001.phpt Zend/tests/hint/param_type_hint_002.phpt Zend/tests/hint/param_type_hint_003.phpt Zend/tests/hint/param_type_hint_004.phpt Zend/t |
||
| References: | 1 2 3 4 | Groups: | php.cvs php.internals |
| Request: | Send a blank email to internals+get-48387@lists.php.net to get a copy of this message | ||
could you discuss that on internals please?
On Sat, May 22, 2010 at 5:15 PM, Ilia Alshanetsky <ilia@prohost.org> wrote:
> Felipe,
>
> The idea behind the more complex logic was to allow people to have
> Int/String/etc... class names, which if we make the types reserved keywords
> will no longer be possible and break existing code.
>
> On Sat, May 22, 2010 at 11:03 AM, Felipe Pena <felipensp@gmail.com> wrote:
>
>> 2010/5/22 Ilia Alshanetsky <ilia@prohost.org>
>>
>> Felipe,
>>>
>>> Wouldn't the simpler change be to allow use of no spaces between scalar
>>> hint and a variable? Basically change + to a *?
>>>
>>>
>>>
>> But it is not just about 0 or more tab/spaces, there is also \r, \n...
>> which are inclued in the {WHITESPACE}.
>> And I consider simpler to match just the type name, thus not needing the
>> yyless() logic. Hence I also suggested in the patch to remove the _HINT
>> suffix.
>>
>> --
>> Regards,
>> Felipe Pena
>>
>
--
Pierre
@pierrejoye | http://blog.thepimp.net | http://www.libgd.org