Re: [PHP-DEV] New gc_status() fields, and WeakMap cycle collection

2023-07-07 Thread Arnaud Le Blanc
I agree that we may not consider this a BC break, but I'm asking in case people rely on the current behavior or if the change is more controversial than I thought. To be fair they work as specified in the RFC, and are actually weak as they do not explicitly retain their keys, but they need explici

Re: [PHP-DEV] New gc_status() fields, and WeakMap cycle collection

2023-07-07 Thread Larry Garfield
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 th

[PHP-DEV] New gc_status() fields, and WeakMap cycle collection

2023-07-07 Thread Arnaud Le Blanc
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 sec