Skip to content
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.

Sign up for GitHub

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

Jump to bottom

lua-imlib2: fix memory leak #2083

Merged
merged 1 commit into from
Nov 18, 2024
Merged

lua-imlib2: fix memory leak #2083

merged 1 commit into from
Nov 18, 2024

Conversation

simotek
Copy link
Contributor

@simotek simotek commented Nov 18, 2024

If you frequently use this function to draw new images this leak becomes quite significant. I use this function for a "Photo Frame" that shows a new photo every 30 seconds, without this change the image was leaked once each time, with this change that leak is resolved.

Checklist

  • [ X] I have described the changes
  • [X ] I have linked to any relevant GitHub issues, if applicable
  • [ X] Documentation in doc/ has been updated
  • [X ] All new code is licensed under GPLv3

Description

  • Describe the changes, why they were necessary, etc
  • Describe how the changes will affect existing behaviour.
  • Describe how you tested and validated your changes.
  • Include any relevant screenshots/evidence demonstrating that the changes work and have been tested.

@simotek
lua-imlib2: fix memory leak
3fb7684 
If you frequently use this function to draw new images this leak
becomes quite significant
@netlify Netlify
Copy link

netlify bot commented Nov 18, 2024 •
edited
Loading

✅ Deploy Preview for conkyweb canceled.

Name Link
🔨 Latest commit 3fb7684
🔍 Latest deploy log https://app.netlify.com/sites/conkyweb/deploys/673a9db8dd233900083c194a

@github-actions github-actions bot added lua related to Lua integration in conky cairo related to cairo (2D graphics library) labels Nov 18, 2024
@simotek simotek mentioned this pull request Nov 18, 2024
Open
@brndnmtthws brndnmtthws added the bug related to incorrect existing implementation of some functionality label Nov 18, 2024
@brndnmtthws brndnmtthws merged commit 9449818 into brndnmtthws:main Nov 18, 2024
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brndnmtthws brndnmtthws brndnmtthws approved these changes

Assignees
No one assigned
Labels
bug related to incorrect existing implementation of some functionality cairo related to cairo (2D graphics library) lua related to Lua integration in conky
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@simotek @brndnmtthws