Hi! > Keeping inconsistent behavior isn't good as well.
It's not good, but it's not also very bad. Nobody really does that, so whatever happens in these cases is mostly irrelevant. Did you test performance impact of those changes? > Actually, $this may be assigned to local variable in run-time only in 3 > places: through $$, extract() and parse_str()/1. For now, maybe. But every time we do something with local symbol table, we'd have to do special case for this. E.g., throretically, what if we wanted to add function to Closure that allows to access to captured scope - we'd need to special-case $this there. Etc... > If we remove this checks, we go back to 7.0 behavior - we create a local > variable $this, that has nothing in common with real internal $this > stored in EX(This). That's not good, but maybe we could make them agree in some way? > In most cases it doesn't show $this in 7.0. Right. But you're changing 7.0 behavior anyway? -- Stas Malyshev smalys...@gmail.com -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
