Hi Dmitry,

On Mon, 30 Aug 2021, Dmitry Stogov wrote:

> Please, let me know you decision according
> https://github.com/derickr/timelib/pull/99
> 
> This workaround fix makes ~170 times improvement on "new DateTimeZone()"
> and as result visible improvement on some real-life apps (e.g Symfony demo
> gets ~7% according to callgrind).
> This is a huge difference.
> 
> The fix was proposed more than a half year ago...
> It would be great to include it into PHP-8.1 release.

I'd seen the original PR, which still had "changes requested" open, but 
I've had a look now and instead of picking an arbitrary limit, we're 
actually respecting the POSIX system limit (which is 6, usually).

Merged into PHP's master (among another bug fix) too.

cheers,
Derick

-- 
PHP 7.4 Release Manager
Host of PHP Internals News: https://phpinternals.news
Like Xdebug? Consider supporting me: https://xdebug.org/support
https://derickrethans.nl | https://xdebug.org | https://dram.io
twitter: @derickr and @xdebug

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: https://www.php.net/unsub.php

Reply via email to