[Telepathy-commits] [telepathy-glib/master] tp_cm_param_setter_offset: use g_value_dup_boxed so we don't need an extra GStrv variable

Guillaume Desmottes guillaume.desmottes at collabora.co.uk
Wed Mar 25 08:58:31 PDT 2009


---
 telepathy-glib/base-connection-manager.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/telepathy-glib/base-connection-manager.c b/telepathy-glib/base-connection-manager.c
index b2b798e..eeb7549 100644
--- a/telepathy-glib/base-connection-manager.c
+++ b/telepathy-glib/base-connection-manager.c
@@ -531,14 +531,13 @@ tp_cm_param_setter_offset (const TpCMParamSpec *paramspec,
             case DBUS_TYPE_STRING:
               {
                 GStrv *save_to = (GStrv *) (params_mem + paramspec->offset);
-                GStrv strv = g_value_get_boxed (value);
 
                 g_strfreev (*save_to);
-                *save_to = g_strdupv (strv);
+                *save_to = g_value_dup_boxed (value);
 
                 if (DEBUGGING)
                   {
-                    gchar *joined = g_strjoinv (", ", strv);
+                    gchar *joined = g_strjoinv (", ", *save_to);
 
                     DEBUG ("%s = [%s]", paramspec->name, joined);
                     g_free (joined);
-- 
1.5.6.5




More information about the telepathy-commits mailing list