Remove support for old D-Bus API
authorAlberto Mardegan <alberto.mardegan@canonical.com>
Fri, 13 Apr 2012 06:52:29 +0000 (09:52 +0300)
committerAlberto Mardegan <alberto.mardegan@canonical.com>
Fri, 13 Apr 2012 06:52:29 +0000 (09:52 +0300)
libsignon-glib/signon-identity-info.c

index c0996f7..2a16381 100644 (file)
@@ -81,14 +81,6 @@ void signon_identity_info_set_methods (SignonIdentityInfo *info,
 }
 
 static void
-identity_value_to_stringarray (gpointer key, gpointer value, gpointer user_data)
-{
-    gchar **stringarray = (gchar **)g_value_get_boxed ((const GValue *)value);
-    g_hash_table_insert ((GHashTable *)user_data, g_strdup((gchar *)key),
-                         g_strdupv (stringarray));
-}
-
-static void
 add_method (gpointer key, gpointer value, gpointer user_data)
 {
     gchar **stringarray = (gchar **)value;
@@ -163,27 +155,14 @@ signon_identity_info_new_from_hash_table (GHashTable *map)
     if (value != NULL)
     {
         g_assert (G_VALUE_HOLDS_BOXED (value));
+        g_assert (G_VALUE_TYPE (value) ==
+                  dbus_g_type_get_map ("GHashTable", G_TYPE_STRING, G_TYPE_STRV));
 
         info->methods = g_hash_table_new_full (g_str_hash, g_str_equal,
                                                g_free, (GDestroyNotify)g_strfreev);
-        /* TODO: this check is needed only for a (hopefully short) transitional period,
-         * until signond has been updated to remove the unnecessary wrapping of the
-         * mechanisms into a D-Bus variant.
-         * Once that is done, the else branch can be removed.
-         */
-        if (G_VALUE_TYPE (value) == dbus_g_type_get_map ("GHashTable",
-                                                         G_TYPE_STRING, G_TYPE_STRV))
-        {
-            g_hash_table_foreach ((GHashTable *)g_value_get_boxed(value),
-                                  add_method,
-                                  info->methods);
-        }
-        else
-        {
-            g_hash_table_foreach ((GHashTable *)g_value_get_boxed(value),
-                                  identity_value_to_stringarray,
-                                  info->methods);
-        }
+        g_hash_table_foreach ((GHashTable *)g_value_get_boxed(value),
+                              add_method,
+                              info->methods);
     }
 
     /* get the accessControlList (gchar**) */