Re: Eligibility of array_key_exists() optimization for 7.3

On Wed, Aug 8, 2018 at 12:50 PM Sara Golemon <pollita@php.net> wrote: > > On Wed, Aug 8, 2018 at 12:11 PM, Michael Moravec <php.net@majkl578.cz> wrote: > > Do you think this optimization could still go into 7.3 so the PHP ecosystem > > could benefit from this improvement (possibly could be reverted in case of > > issues)? > > > My vote is no. > > It's an optimization and optimizations are yay, but feature freeze > isn't just a PR label, it's a release process milestone. The tradeoff > against code-stability just isn't there. > > -Sara > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php I agree with Sara. Sadly, I also have some similar improvements I wanted to get in for improving the performance of array_slice but couldn't meet feature freeze. If we were to bring in my changes and yours and who knows who else may have lurking changes like this then the stability risk Sara mentioned is real. It hurts to wait for another release but the bright side is that is a lot of time for the feature to get tested and be far less risky.