diff options
| author | Luiz Augusto von Dentz <luiz.von.dentz@intel.com> | 2013-02-27 14:00:23 +0200 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2013-04-08 20:10:39 -0700 |
| commit | 3d6483cd7dd585b7673b62ffd418d3cab623f128 (patch) | |
| tree | 2fb5dd7b33763b9b5f660374c6a58b2bf6399a10 | |
| parent | a57b8b7fe02344417e0f957ee777db72e644321f (diff) | |
| download | mmsd-3d6483cd7dd585b7673b62ffd418d3cab623f128.tar.gz | |
gdbus: Fix not calling property_changed callback
In case a property did not appear by the time proxy_added was called
property_changed has to be called if it appear latter otherwise the
application will be unaware of it.
| -rw-r--r-- | gdbus/client.c | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/gdbus/client.c b/gdbus/client.c index 369e3ac..2a7d2e1 100644 --- a/gdbus/client.c +++ b/gdbus/client.c @@ -205,6 +205,7 @@ static void prop_entry_free(gpointer data) static void add_property(GDBusProxy *proxy, const char *name, DBusMessageIter *iter, gboolean send_changed) { + GDBusClient *client = proxy->client; DBusMessageIter value; struct prop_entry *prop; @@ -215,20 +216,8 @@ static void add_property(GDBusProxy *proxy, const char *name, prop = g_hash_table_lookup(proxy->prop_list, name); if (prop != NULL) { - GDBusClient *client = proxy->client; - prop_entry_update(prop, &value); - - if (proxy->prop_func) - proxy->prop_func(proxy, name, &value, proxy->prop_data); - - if (client == NULL || send_changed == FALSE) - return; - - if (client->property_changed) - client->property_changed(proxy, name, &value, - client->user_data); - return; + goto done; } prop = prop_entry_new(name, &value); @@ -237,8 +226,16 @@ static void add_property(GDBusProxy *proxy, const char *name, g_hash_table_replace(proxy->prop_list, prop->name, prop); +done: if (proxy->prop_func) proxy->prop_func(proxy, name, &value, proxy->prop_data); + + if (client == NULL || send_changed == FALSE) + return; + + if (client->property_changed) + client->property_changed(proxy, name, &value, + client->user_data); } static void update_properties(GDBusProxy *proxy, DBusMessageIter *iter, |