Code review comment for lp:~mterry/indicator-messages/tell-accounts-services

Revision history for this message
Charles Kerr (charlesk) wrote :

The patch is convoluted wrt stacking dbus calls, but it's probably unavoidable for this task. Except for some superficial changes like using g_clear_object(), I would probably write it about the same... :)

review: Approve

« Back to merge proposal