[Bug 28602] Should be ported to GSettings

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Aug 2 10:24:09 CEST 2010


https://bugs.freedesktop.org/show_bug.cgi?id=28602

Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|patch                       |

--- Comment #7 from Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> 2010-08-02 01:24:07 PDT ---
+  if (priv->gsettings != NULL)
     {
-      g_object_unref (priv->client);
-      priv->client = NULL;
+      g_object_unref (priv->gsettings);
+      priv->gsettings = NULL;

tp_clear_object() ftw!


http://git.collabora.co.uk/?p=user/danni/telepathy-logger.git;a=commitdiff;h=30b0ce878d6cb3837b9cd207eb98f873ed553dbf
Why not use the extra magic generating the list file?

If you add it, I won't revert it, it's always something useful to have.

TplConf seems to have turned to an extra useless layer on top of the GSettings
object. Shouldn't we get rid of it?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.


More information about the telepathy-bugs mailing list