Skip to content

Image loading by file name memory leak is not properly fixed #2716

Description

@homm

__del__ doesn't work, because in the most cases self.fp is None after initial loading and actual fp pointer is stored in self.__fp

Activity

  1. added
    BugAny unexpected behavior, until confirmed feature.
    on Apr 1, 2018
  2. added this to the Future milestone on Apr 1, 2018
  3. radarhere commented on Sep 7, 2018

    @radarhere
    Member

    I have created PR #3261 to try and address this.

  4. radarhere commented on Dec 13, 2018

    @radarhere
    Member

    @homm is this now fixed on master?

  5. aclark4life commented on May 11, 2019

    @aclark4life
    Member

    @hugovk @radarhere If you still think this can go in and no objection from @wiredfool or @homm I'd probably just merge it. (I see the PR was merged.)

  6. hugovk commented on May 12, 2019

    @hugovk
    Member

    The PR is merged, closing this issue let's re-open or create a new issue needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

BugAny unexpected behavior, until confirmed feature.File ClosingMemory

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions