Closed zappolowski closed 4 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
8196755
) 64.48% compared to head (9a54a70
) 65.22%.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Could you explain the memory leak?
From what I've seen it comes from g_variant_new_from_bytes
(which is not necessary here and can be avoided by using typed versions like I did) and g_variant_type_new
, which even states, that it transfers ownership and thus should be freed.
I've added some minor clean up:
builder
- just leave it on the stack as management is easier this wayanswer
just for conciseness - g_dbus_method_invocation_return_value
takes care of the floating variant created by g_variant_new
alreadyPS: I cannot reproduce the pipeline failure. That image ran successfully on my machine. Pushed again without changes ... now it works.
@fwsmit If you agree with this, I'd like to include the fixup into the first commit before merging.
looks good. Are you done with the changes or something is still pending?
Also friso is away at the moment
... and while at it also fix all memory leaks from this method.