Re: New gc_status() fields, and WeakMap cycle collection
| From: | Larry Garfield | Date: | Fri, 07 Jul 2023 14:02:50 +0000 |
| Subject: | Re: New gc_status() fields, and WeakMap cycle collection | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-120763@lists.php.net to get a copy of this message | ||
On Fri, Jul 7, 2023, at 1:09 PM, Arnaud Le Blanc wrote:
> Hi everyone,
>
> I'm considering merging the following two PRs:
>
> https://github.com/php/php-src/pull/11523: Exposes the time
> spent in
> the cycle collector via new gc_status() fields.
> https://github.com/php/php-src/pull/10932: Allows the GC to
> collect
> certain types of cycles involving WeakMaps.
>
> The second one introduces a minor BC break.
>
> If there are no objections, I plan to merge these PRs prior to the
> feature freeze.
>
> Best regards,
> Arnaud
For the second, doesn't that translate to "WeakMaps are currently not actually Weak, and
thus don't work at all?" I don't quite know how that would be a BC break to fix...
--Larry Garfield