Skip to content

bpo-30776: regrtest: reduce memleak false positive #2484

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
Jun 29, 2017
Merged

bpo-30776: regrtest: reduce memleak false positive #2484

merged 1 commit into from
Jun 29, 2017

Conversation

vstinner
Copy link
Member

Only report a leak if each run leaks at least one memory block.

Only report a leak if each run leaks at least one memory block.
@mention-bot
Copy link

@Haypo, thanks for your PR! By analyzing the history of the files in this pull request, we identified @serhiy-storchaka and @ilevkivskyi to be potential reviewers.

@vstinner
Copy link
Member Author

This PR is similar to my commit 48b5c42 which helped to avoid false positive reference leaks.

@vstinner vstinner requested a review from pitrou June 29, 2017 07:43
@vstinner
Copy link
Member Author

I consider that the following results are false positive:

test_import leaked [0, 0, 2] memory blocks, sum=2
test_import leaked [-2, 0, 0] memory blocks, sum=-2
test_tools leaked [0, 0, 6] memory blocks, sum=6

http://buildbot.python.org/all/builders/AMD64%20Windows8.1%20Refleaks%203.x/builds/37/steps/test/logs/stdio

@vstinner vstinner merged commit beeca6e into python:master Jun 29, 2017
@vstinner vstinner deleted the memleak branch June 29, 2017 08:32
vstinner added a commit that referenced this pull request Jun 30, 2017
…r to 3.6 (#2513)

* bpo-29512: Add test.bisect, bisect failing tests (#2452)

Add a new "python3 -m test.bisect" tool to bisect failing tests.

It can be used to find which test method(s) leak references, leak
files, etc.
(cherry picked from commit 84d9d14)

* bpo-30776: regrtest: reduce memleak false positive (#2484)

Only report a leak if each run leaks at least one memory block.
(cherry picked from commit beeca6e)
vstinner added a commit that referenced this pull request Jul 3, 2017
… from 3.6 to 3.5 (#2540)

* [3.6] bpo-29512, bpo-30776: Backport regrtest enhancements from master to 3.6 (#2513)

* bpo-29512: Add test.bisect, bisect failing tests (#2452)

Add a new "python3 -m test.bisect" tool to bisect failing tests.

It can be used to find which test method(s) leak references, leak
files, etc.
(cherry picked from commit 84d9d14)

* bpo-30776: regrtest: reduce memleak false positive (#2484)

Only report a leak if each run leaks at least one memory block.
(cherry picked from commit beeca6e)

(cherry picked from commit a3ca94d)

* bpo-30764: Fix regrtest --fail-env-changed --forever (#2536) (#2539)

--forever now stops if a fail changes the environment.
(cherry picked from commit 5e87592)
(cherry picked from commit 4132adb)
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