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: Date: Sat, 22 May 2010 15:15:24 +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  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-62687@lists.php.net to get a copy of this message
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 >

« previous php.cvs (#62687) next »