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

GVariant, GError and g_autoptr cleanups #1439

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

GVariant, GError and g_autoptr cleanups #1439

wants to merge 4 commits into from
+79 −75

Conversation

swick
Copy link
Contributor

@swick swick commented Sep 23, 2024

No description provided.

whot
whot approved these changes Oct 4, 2024
View reviewed changes
Copy link
Contributor

@whot whot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about the non-floating gvariants patch but it looks correct and the rest too.

src/input-capture.c Show resolved Hide resolved
@swick
Use g_autoptr for GErrors in more places
7bc435a
@swick
realtime: Use g_autoptr for non-floating GVariants
ac5f7d8
@swick
Always create an empty vardict instead of passing NULL
2c9a981 
In the error case, we will clear the GVariant and then try to emit a
response from a NULL pointer as result. Always create an empty vardict
when something failed instead.

This also clears the error because we continue on in the error case and
do in fact re-using it in one case.
@swick
portals: Work with owned, non-floating GVariants where possible
543178d 
By g_variant_ref_sink'ing floating GVariants as early as possible. This
fixes a few cases where the variant would be leaked, where the floating
ref would be owned by a g_autoptr and makes it easier to see who owns
the reference.

Cases where the floating ref is passed into a function which immediately
sinks the floating ref are left untouched.
@swick
Copy link
Contributor Author

swick commented Oct 4, 2024

I'm convinced that using floating references is in general very dangerous and with g_auto there isn't much of a use for them. The only place where not sinking the floating reference immediately is acceptable in my opinion is when it is passed directly as an argument to a function which is clearly documented to sink the reference.

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

@whot whot whot approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Triage
Status: Needs Triage
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@swick @whot