Skip to content

Conversation

renewedvision-kh
Copy link
Contributor

New Pull Request Checklist

  • I have read and understood the CONTRIBUTING guide

  • I have read the Documentation

  • I have searched for a similar pull request in the project and found none

  • I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

  • I have added the required tests to prove the fix/feature I am adding

  • I have updated the documentation (if necessary)

  • I have run the tests and they pass

  • I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: ...
#3776

Pull Request Description

Adds an @autoreleasepool to the DisplayLinkCallback to release any objects added to an AutoreleasePool during the callback thereby avoiding leaking the objects until the CVDisplayLink is closed.
...

@dreampiggy dreampiggy added leak Memory Leak macOS AppKit for macOS labels Jan 6, 2025
@dreampiggy dreampiggy merged commit 15f6675 into SDWebImage:master Jan 16, 2025
5 of 7 checks passed
@dreampiggy dreampiggy added this to the 5.20.1 milestone Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
leak Memory Leak macOS AppKit for macOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants