Merge lp:~ken-vandine/indicator-me/lp_754524 into lp:indicator-me

Proposed by Ken VanDine
Status: Merged
Approved by: Ted Gould
Approved revision: 146
Merged at revision: 145
Proposed branch: lp:~ken-vandine/indicator-me/lp_754524
Merge into: lp:indicator-me
Diff against target: 30 lines (+3/-3)
2 files modified
src/indicator-me.c (+1/-1)
src/me-service-dbus.c (+2/-2)
To merge this branch: bzr merge lp:~ken-vandine/indicator-me/lp_754524
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
Review via email: mp+57713@code.launchpad.net

Description of the change

Pass the username as a g_variant on the UserChanged signal (LP: #754524)

To post a comment you must log in.
Revision history for this message
Ted Gould (ted) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/indicator-me.c'
--- src/indicator-me.c 2011-04-04 16:47:54 +0000
+++ src/indicator-me.c 2011-04-14 15:46:27 +0000
@@ -179,7 +179,7 @@
179username_cb (GObject * object, GAsyncResult * res, gpointer user_data)179username_cb (GObject * object, GAsyncResult * res, gpointer user_data)
180{180{
181 GError * error = NULL;181 GError * error = NULL;
182 gchar * username;182 gchar * username = NULL;
183 GVariant * result;183 GVariant * result;
184184
185 IndicatorMe * self = INDICATOR_ME (user_data);185 IndicatorMe * self = INDICATOR_ME (user_data);
186186
=== modified file 'src/me-service-dbus.c'
--- src/me-service-dbus.c 2011-01-13 20:37:08 +0000
+++ src/me-service-dbus.c 2011-04-14 15:46:27 +0000
@@ -310,11 +310,11 @@
310 INDICATOR_ME_SERVICE_DBUS_OBJECT,310 INDICATOR_ME_SERVICE_DBUS_OBJECT,
311 INDICATOR_ME_SERVICE_DBUS_INTERFACE,311 INDICATOR_ME_SERVICE_DBUS_INTERFACE,
312 "UserChanged",312 "UserChanged",
313 NULL,313 g_variant_new("(s)", username),
314 &error);314 &error);
315315
316 if (error != NULL) {316 if (error != NULL) {
317 g_error("Unable to send UserChanged signal: %s", error->message);317 g_warning("Unable to send UserChanged signal: %s", error->message);
318 g_error_free(error);318 g_error_free(error);
319 return;319 return;
320 }320 }

Subscribers

People subscribed via source and target branches