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.

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 some memory leaks #1450

Merged
merged 1 commit into from
Oct 6, 2024
Merged

Fix some memory leaks #1450

merged 1 commit into from
Oct 6, 2024

Conversation

hfiguiere
Copy link
Collaborator

g_build_filename() always transfer full
Add auto cleanup for Settings implementation

g_build_filename() always transfer full
Add auto cleanup for Settings implementation

Signed-off-by: Hubert Figuière <[email protected]>
@hfiguiere hfiguiere self-assigned this Oct 6, 2024
@swick
Copy link
Collaborator

swick commented Oct 6, 2024

LGTM

@GeorgesStavracas GeorgesStavracas added this to the 1.20 milestone Oct 6, 2024
@GeorgesStavracas GeorgesStavracas added the portal: settings Settings portal label Oct 6, 2024
@GeorgesStavracas GeorgesStavracas added this pull request to the merge queue Oct 6, 2024
Merged via the queue into flatpak:main with commit d1bc22f Oct 6, 2024
4 checks passed
@hfiguiere hfiguiere deleted the leaks branch October 6, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
portal: settings Settings portal
Projects
No open projects
Status: Triaged
Development

Successfully merging this pull request may close these issues.

3 participants