Skip to content

Fix memory leak from #154 #156

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 24, 2019
Merged

Fix memory leak from #154 #156

merged 1 commit into from
Mar 24, 2019

Conversation

dnna
Copy link
Contributor

@dnna dnna commented Mar 24, 2019

Fixes the leak described in #154

Memory before the fix:

Memory Usage: 11437000
Memory Usage: 11429088
Memory Usage: 11429808
Memory Usage: 11430528
Memory Usage: 11431248
Memory Usage: 11431968
Memory Usage: 11432688
Memory Usage: 11433408
Memory Usage: 11434128
Memory Usage: 11434848
Memory Usage: 11435568
Memory Usage: 11436288
Memory Usage: 11437008
Memory Usage: 11437728
Memory Usage: 11438448
Memory Usage: 11439168
Memory Usage: 11439888
Memory Usage: 11440608
Memory Usage: 11441328
Memory Usage: 11442048

Memory after the fix:

Memory Usage: 11483624
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992
Memory Usage: 11474992

@andig andig merged commit 057f02e into php-pm:master Mar 24, 2019
@andig
Copy link
Contributor

andig commented Mar 24, 2019

I've changed the name of this commit, still GREAT!!!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants