# Small GC optimization

- Thread: https://externals.io/message/69944
- Original archive: https://news-web.php.net/php.internals/69944
- Started by Dmitry Stogov on 2013-10-29T17:29:51+00:00
- Messages: 2
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #69944 — Dmitry Stogov — 2013-10-29T17:29:51+00:00

URL: https://externals.io/message/69944#69944

Hi,

The proposed patch for PHP master branch eliminates a number of calls
related to Garbage Collector.

https://gist.github.com/dstogov/7218890

I don't see any failed PHPT tests and I can't imagine a script that may
"leak" after the change, but I can't be 100% sure that I didn't miss
anything.

I would appreciate, if anyone who has their own test suites, run them with
patched DEBUG build. Please email, if you get any memory leak related
errors.

Thanks. Dmitry.

---

## #69946 — Joe Watkins — 2013-10-29T18:15:30+00:00

URL: https://externals.io/message/69944#69946 · In reply to: #69944

> Hi,
>
> The proposed patch for PHP master branch eliminates a number of calls
> related to Garbage Collector.
>
> https://gist.github.com/dstogov/7218890
>
> I don't see any failed PHPT tests and I can't imagine a script that may
> "leak" after the change, but I can't be 100% sure that I didn't miss
> anything.
>
> I would appreciate, if anyone who has their own test suites, run them with
> patched DEBUG build. Please email, if you get any memory leak related
> errors.
>
> Thanks. Dmitry.
>

Niiiice ...

I really expected that'd cause pthreads some trouble, no failures to 
report :)

Cheers
Joe