From 4b3679b550a10d4e18097c7278e876c726de9e9d Mon Sep 17 00:00:00 2001 From: scorpion-26 Date: Sun, 5 Nov 2023 22:10:02 +0100 Subject: [PATCH] Cleanup commented out g_variant_unref calls They were bogus and the added comment explains why they're not needed in this case --- src/SNI.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/SNI.cpp b/src/SNI.cpp index c5a3498..29f9d51 100644 --- a/src/SNI.cpp +++ b/src/SNI.cpp @@ -144,10 +144,7 @@ namespace SNI g_error_free(err); return nullptr; } - // There's probably a better method than to use 3 variants - // g_variant_unref(params[0]); - // g_variant_unref(params[1]); - // g_variant_unref(param); + // g_dbus_connection_call_sync consumes the parameter and its children, so no need to unref return res; };