Skip to content

[2.7] bpo-36560: Fix reference leak hunting in regrtest (GH-12744) #12745

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 9, 2019
Merged

[2.7] bpo-36560: Fix reference leak hunting in regrtest (GH-12744) #12745

merged 1 commit into from
Apr 9, 2019

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Apr 9, 2019

Fix reference leak hunting in regrtest: compute also deltas (of
reference count, allocated memory blocks, file descriptor count)
during warmup, to ensure that everything is initialized before
starting to hunt reference leaks.

Other changes:

  • Replace gc.collect() with support.gc_collect() in clear_caches()
  • dash_R() is now more quiet with --quiet option (don't display
    progress).
  • Precompute the full range for "for it in range(repcount):" to
    ensure that the iteration doesn't allocate anything new.
  • dash_R() now is responsible to call warm_caches().

(cherry picked from commit 5aaac94)

https://bugs.python.org/issue36560

Fix reference leak hunting in regrtest: compute also deltas (of
reference count and file descriptor count) during warmup, to ensure
that everything is initialized before starting to hunt reference
leaks.

Other changes:

* Replace gc.collect() with support.gc_collect() in clear_caches()
* dash_R() is now more quiet with --quiet option (don't display
  progress).
* Precompute the full range for "for it in range(repcount):" to
  ensure that the iteration doesn't allocate anything new.
* dash_R() now is responsible to call warm_caches().

(cherry picked from commit 5aaac94)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants