> De : Pierre Joye [mailto:pierre....@gmail.com] > > As I said earlier, E_DEPRECATED will only caught what it sees, we miss > what it does not and actually casts happily.
Sorry, Pierre, I don't see what you mean. You're probably right but I don't understand what you mean with 'casting'. AFAIK, we are not touching casting rules, implicit or explicit. We just take the ZPP code and raise some E_DEPRECATED while keeping the whole logic. Can you try to explain again or give a scenario I can understand. > > > And temporary because developers will quietly fix their code and the > hidden messages will disappear. So, we probably won't have to rely on 'good > luck'. > > Again. It seems you only see issues with this insignificant > E_DEPRECATED messages, it will be just like other or E_NOTICE, they > will disable it and move on. However my point is in a totally > different area, the one we have no idea what is going on because it is > now casted while it was not before. Do you mean that adding type hints will break existing code ? If someone can explain... We do not have data to cover these > cases. It is not a lack of will but it is simply impossible to have > for all apps out there in production (as you said before). And it is > exactly why I am against changing the casting rules, even for a single > yota beyond what we have done already (or close to this area). > > > > FUD apart, every test Dmitry ran using his upcoming patch (which will > implement E_DEPRECATED) on existing PHP software raised very few new > errors. More : after analysis, all these messages except 1, I believe, > correspond to undetected bugs in the PHP code. Like the bug I detected in > the PHP code to build phar.phar. So, I can go further : we are not breaking > anything *and* we are helping users to find undetected bugs in their > codebase. Nice side effect, isn't it ? > > Again, see my previous comment in this mail. > > Cheers, > -- > Pierre > > @pierrejoye | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
