From 94398e25e2d4bc74cee310c50c25bf129ee9625e Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Thu, 16 Feb 2012 15:37:03 -0600 Subject: register for g-properties-changed rather than g-signal --- src/indicator-power.c | 34 +++++++++++++++------------------- 1 file changed, 15 insertions(+), 19 deletions(-) diff --git a/src/indicator-power.c b/src/indicator-power.c index 3980988..a5092c1 100644 --- a/src/indicator-power.c +++ b/src/indicator-power.c @@ -840,26 +840,22 @@ update_visibility (IndicatorPower * self) } static void -receive_signal (GDBusProxy *proxy, - gchar *sender_name, - gchar *signal_name, - GVariant *parameters, - gpointer user_data) +receive_properties_changed (GDBusProxy *proxy G_GNUC_UNUSED, + GVariant *changed_properties G_GNUC_UNUSED, + GStrv invalidated_properties G_GNUC_UNUSED, + gpointer user_data) { IndicatorPower *self = INDICATOR_POWER (user_data); - if (g_strcmp0 (signal_name, "Changed") == 0) - { - /* get the new state */ - g_dbus_proxy_call (self->proxy, - "GetDevices", - NULL, - G_DBUS_CALL_FLAGS_NONE, - -1, - self->proxy_cancel, - get_devices_cb, - user_data); - } + /* it's time to refresh our device list */ + g_dbus_proxy_call (self->proxy, + "GetDevices", + NULL, + G_DBUS_CALL_FLAGS_NONE, + -1, + self->proxy_cancel, + get_devices_cb, + user_data); } static void @@ -884,8 +880,8 @@ service_proxy_cb (GObject *object, /* we want to change the primary device changes */ g_signal_connect (self->proxy, - "g-signal", - G_CALLBACK (receive_signal), + "g-properties-changed", + G_CALLBACK (receive_properties_changed), user_data); /* get the initial state */ -- cgit v1.2.3 From 947e415ae432d1f0ec0d95955c0d8a71323eb4c1 Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Wed, 22 Feb 2012 15:25:29 -0600 Subject: fix set_accessible_desc() to update the entries' accessible_desc fields and emit the accessible-desc-changed signal --- src/indicator-power.c | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/src/indicator-power.c b/src/indicator-power.c index a5092c1..d3d929f 100644 --- a/src/indicator-power.c +++ b/src/indicator-power.c @@ -402,12 +402,23 @@ static void set_accessible_desc (IndicatorPower *self, const gchar *desc) { - if (desc == NULL || desc[0] == '\0') - return; - - g_free (self->accessible_desc); - - self->accessible_desc = g_strdup (desc); + if (desc && *desc) + { + /* update our copy of the string */ + char * old_desc = self->accessible_desc; + self->accessible_desc = g_strdup (desc); + g_free (old_desc); + + /* have the entries use our string */ + GList * l; + GList * entries = indicator_object_get_entries(INDICATOR_OBJECT(self)); + for (l=entries; l!=NULL; l=l->next) { + IndicatorObjectEntry * entry = l->data; + entry->accessible_desc = self->accessible_desc; + g_signal_emit (self, INDICATOR_OBJECT_SIGNAL_ACCESSIBLE_DESC_UPDATE_ID, 0, entry); + } + g_list_free (entries); + } } static const gchar * -- cgit v1.2.3 From f688c48867bc6d579ed1071b1e58a4afeb0bb1d5 Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Wed, 22 Feb 2012 15:27:56 -0600 Subject: fix trivial misspelling --- src/indicator-power.c | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/src/indicator-power.c b/src/indicator-power.c index d3d929f..9b2568e 100644 --- a/src/indicator-power.c +++ b/src/indicator-power.c @@ -329,7 +329,7 @@ build_device_time_details (const gchar *device_name, gdouble percentage, gchar **short_details, gchar **details, - gchar **accesible_name) + gchar **accessible_name) { gchar *short_timestring = NULL; gchar *detailed_timestring = NULL; @@ -343,8 +343,8 @@ build_device_time_details (const gchar *device_name, if (state == UP_DEVICE_STATE_CHARGING) { /* TRANSLATORS: %2 is a time string, e.g. "1 hour 5 minutes" */ - *accesible_name = g_strdup_printf (_("%s (%s to charge (%.0lf%%))"), - device_name, detailed_timestring, percentage); + *accessible_name = g_strdup_printf (_("%s (%s to charge (%.0lf%%))"), + device_name, detailed_timestring, percentage); *details = g_strdup_printf (_("%s (%s to charge)"), device_name, short_timestring); *short_details = g_strdup_printf ("(%s)", short_timestring); @@ -355,14 +355,14 @@ build_device_time_details (const gchar *device_name, if (time > 43200) /* 12 hours */ { - *accesible_name = g_strdup_printf (_("%s"), device_name); + *accessible_name = g_strdup_printf (_("%s"), device_name); *details = g_strdup_printf (_("%s"), device_name); } else { /* TRANSLATORS: %2 is a time string, e.g. "1 hour 5 minutes" */ - *accesible_name = g_strdup_printf (_("%s (%s left (%.0lf%%))"), - device_name, detailed_timestring, percentage); + *accessible_name = g_strdup_printf (_("%s (%s left (%.0lf%%))"), + device_name, detailed_timestring, percentage); *details = g_strdup_printf (_("%s (%s left)"), device_name, short_timestring); } @@ -376,7 +376,7 @@ build_device_time_details (const gchar *device_name, if (state == UP_DEVICE_STATE_FULLY_CHARGED) { *details = g_strdup_printf (_("%s (charged)"), device_name); - *accesible_name = g_strdup (*details); + *accessible_name = g_strdup (*details); *short_details = g_strdup (""); } else if (percentage > 0) @@ -385,14 +385,14 @@ build_device_time_details (const gchar *device_name, * used when we don't have a time value */ *details = g_strdup_printf (_("%s (%.0lf%%)"), device_name, percentage); - *accesible_name = g_strdup (*details); + *accessible_name = g_strdup (*details); *short_details = g_strdup_printf (_("(%.0lf%%)"), percentage); } else { *details = g_strdup_printf (_("%s (not present)"), device_name); - *accesible_name = g_strdup (*details); + *accessible_name = g_strdup (*details); *short_details = g_strdup (_("(not present)")); } } @@ -533,7 +533,7 @@ menu_add_device (GtkMenu *menu, const gchar *device_name; gchar *short_details = NULL; gchar *details = NULL; - gchar *accesible_name = NULL; + gchar *accessible_name = NULL; if (device == NULL) return; @@ -559,7 +559,7 @@ menu_add_device (GtkMenu *menu, device_name = device_kind_to_localised_string (kind); - build_device_time_details (device_name, time, state, percentage, &short_details, &details, &accesible_name); + build_device_time_details (device_name, time, state, percentage, &short_details, &details, &accessible_name); /* Create menu item */ item = gtk_image_menu_item_new (); @@ -579,7 +579,7 @@ menu_add_device (GtkMenu *menu, g_free (short_details); g_free (details); - g_free (accesible_name); + g_free (accessible_name); g_free (device_icon); g_free (object_path); } @@ -757,7 +757,7 @@ put_primary_device (IndicatorPower *self, GIcon *device_gicons; gchar *short_details = NULL; gchar *details = NULL; - gchar *accesible_name = NULL; + gchar *accessible_name = NULL; gchar *device_icon = NULL; gchar *object_path = NULL; gdouble percentage; @@ -788,15 +788,15 @@ put_primary_device (IndicatorPower *self, device_name = device_kind_to_localised_string (kind); /* get the description */ - build_device_time_details (device_name, time, state, percentage, &short_details, &details, &accesible_name); + build_device_time_details (device_name, time, state, percentage, &short_details, &details, &accessible_name); gtk_label_set_label (GTK_LABEL (self->label), short_details); - set_accessible_desc (self, accesible_name); + set_accessible_desc (self, accessible_name); g_free (short_details); g_free (details); - g_free (accesible_name); + g_free (accessible_name); g_free (device_icon); g_free (object_path); } -- cgit v1.2.3 From 92c017505705db9f224f50a536dd5d7ca06c9bea Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Wed, 22 Feb 2012 15:44:05 -0600 Subject: fix trivial indentation error --- src/indicator-power.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/indicator-power.c b/src/indicator-power.c index 9b2568e..cd691d8 100644 --- a/src/indicator-power.c +++ b/src/indicator-power.c @@ -405,7 +405,7 @@ set_accessible_desc (IndicatorPower *self, if (desc && *desc) { /* update our copy of the string */ - char * old_desc = self->accessible_desc; + char * old_desc = self->accessible_desc; self->accessible_desc = g_strdup (desc); g_free (old_desc); -- cgit v1.2.3 From 5396553577a4ecd2b75dd5a1bb398b5e92a0e076 Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Wed, 22 Feb 2012 17:39:50 -0600 Subject: when a new entry is added, ensure that its accessible_string is properly set --- src/indicator-power.c | 47 +++++++++++++++++++++++++++++++++++++---------- 1 file changed, 37 insertions(+), 10 deletions(-) diff --git a/src/indicator-power.c b/src/indicator-power.c index cd691d8..6140c54 100644 --- a/src/indicator-power.c +++ b/src/indicator-power.c @@ -103,6 +103,8 @@ static const gchar* get_name_hint (IndicatorObject * io); static void update_visibility (IndicatorPower * self); static gboolean should_be_visible (IndicatorPower * self); +static void on_entry_added (IndicatorObject * io, IndicatorObjectEntry * entry, gpointer user_data); + static void gsd_appeared_callback (GDBusConnection *connection, const gchar *name, const gchar *name_owner, gpointer user_data); G_DEFINE_TYPE (IndicatorPower, indicator_power, INDICATOR_OBJECT_TYPE); @@ -144,6 +146,9 @@ indicator_power_init (IndicatorPower *self) g_object_set (G_OBJECT(self), INDICATOR_OBJECT_DEFAULT_VISIBILITY, FALSE, NULL); + + g_signal_connect (INDICATOR_OBJECT(self), INDICATOR_OBJECT_SIGNAL_ENTRY_ADDED, + G_CALLBACK(on_entry_added), NULL); } static void @@ -398,26 +403,48 @@ build_device_time_details (const gchar *device_name, } } +/* ensure that the entry is using self's accessible description */ +static void +refresh_entry_accessible_desc (IndicatorPower * self, IndicatorObjectEntry * entry) +{ + const char * newval = self->accessible_desc; + + if (entry->accessible_desc != newval) + { + g_debug ("%s: setting entry %p accessible description to '%s'", G_STRFUNC, entry, newval); + entry->accessible_desc = newval; + g_signal_emit (self, INDICATOR_OBJECT_SIGNAL_ACCESSIBLE_DESC_UPDATE_ID, 0, entry); + } +} + +static void +on_entry_added (IndicatorObject * io, + IndicatorObjectEntry * entry, + gpointer user_data G_GNUC_UNUSED) +{ + refresh_entry_accessible_desc (INDICATOR_POWER(io), entry); +} + static void -set_accessible_desc (IndicatorPower *self, - const gchar *desc) +set_accessible_desc (IndicatorPower *self, const gchar *desc) { + g_debug ("%s: setting accessible description to '%s'", G_STRFUNC, desc); + if (desc && *desc) { /* update our copy of the string */ - char * old_desc = self->accessible_desc; + char * oldval = self->accessible_desc; self->accessible_desc = g_strdup (desc); - g_free (old_desc); - /* have the entries use our string */ + /* ensure that the entries are using self's accessible description */ GList * l; GList * entries = indicator_object_get_entries(INDICATOR_OBJECT(self)); - for (l=entries; l!=NULL; l=l->next) { - IndicatorObjectEntry * entry = l->data; - entry->accessible_desc = self->accessible_desc; - g_signal_emit (self, INDICATOR_OBJECT_SIGNAL_ACCESSIBLE_DESC_UPDATE_ID, 0, entry); - } + for (l=entries; l!=NULL; l=l->next) + refresh_entry_accessible_desc (self, l->data); + + /* cleanup */ g_list_free (entries); + g_free (oldval); } } -- cgit v1.2.3