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

Fix leak of gc's in gtkagg backend #1485

Merged
merged 1 commit into from Nov 12, 2012

Conversation

Projects
None yet
3 participants
@cmichal2
Contributor

cmichal2 commented Nov 11, 2012

The gtkagg backend creates gc's with gdk_gc_new, but never destroyed them. This commit adds a matching gdk_gc_destroy to fix the memory leak.

@dmcdougall

This comment has been minimized.

Member

dmcdougall commented Nov 12, 2012

Looks good to me.

@mdboom

This comment has been minimized.

Member

mdboom commented Nov 12, 2012

Agreed. Merging. This definitely belongs on the 1.2.x maintenance branch (as you have done). Thanks.

mdboom added a commit that referenced this pull request Nov 12, 2012

Merge pull request #1485 from cmichal2/v1.2.x-gc
Fix leak of gc's in gtkagg backend

@mdboom mdboom merged commit 1513f4a into matplotlib:v1.2.x Nov 12, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment