Zeev, First of all as far as I can tell majority of the changes relate to 2 new type hints you are suggesting to introduce which are numeric and scalar. I don't see any issue with adding those two hints, predominantly for people who don't want to be specific with their type requirements. So, +1 on that portion of the RFC.
However, after reviewing your suggested changes to some of the type-specific hints, I think the changes you are suggesting would only introduce confusion. There is a substantial WTF factor for developers, whereby (int)1 === boolean, but (int)12 !== boolean. Or that (int) 12 is either a valid int or a double, but (double)12 is only a double. For one strict type hints are much clearer, if function/method expects type X and anything but type X is provided an error is generated. I think adding the scalar and numeric hints should be a sufficient. On Sat, May 22, 2010 at 12:09 PM, Zeev Suraski <z...@zend.com> wrote: > FYI - I did some fairly major editing on the issues with strict type > hinting and advantages of auto-converting type hinting in the RFC > (beforehand I focused primarily on the conversion table). > > I encourage everyone to take another look at this RFC even if you've read > it before. > > Thanks, > > Zeev > > > At 18:04 22/05/2010, Zeev Suraski wrote: > >> All, >> >> As I mentioned numerous times scalar type hinting is a very controversial >> feature in my mind and requires some substantial discussion before we move >> ahead to implement it. >> As such, I think we should revert the patch that was committed to trunk >> and move to discuss it first instead. >> >> My (& Lukas') thoughts on the subject are detailed at < >> http://wiki.php.net/rfc/typecheckingstrictandweak> >> http://wiki.php.net/rfc/typecheckingstrictandweak >> >> As one of the key people who designed PHP's type system I consider strict >> type checks completely alien to and counterintuitive in PHP and am therefore >> pushing to implement 'weak' typing instead, in a way that's consistent and >> familiar to users. >> >> Zeev >> >> -- >> PHP Internals - PHP Runtime Development Mailing List >> To unsubscribe, visit: http://www.php.net/unsub.php >> > > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > >
