Re: com php-src: Fixed bug #74878: NEWS Zend/zend_API.c
| From: | Nikita Popov | Date: | Mon, 10 Jul 2017 09:16:08 +0000 |
| Subject: | Re: com php-src: Fixed bug #74878: NEWS Zend/zend_API.c | ||
| References: | 1 2 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-98487@lists.php.net to get a copy of this message | ||
On Mon, Jul 10, 2017 at 10:08 AM, Dmitry Stogov <dmitry@zend.com> wrote:
> This commit introduced a lot of memory leaks.
>
> Probably, you meant string interning only for internal classes.
>
> Even for internal classes only, this may require additional changes in
> opcache, that copies interned strings into shared memory.
>
>
> It's better to revert this, before complete fix.
>
If this is causing issues, please revert it. I think it should always
intern even for non-internal classes (these strings are likely duplicated
compile-time strings, something we usually intern), but I clearly did not
do it correctly.
Nikita