Hi Lucas,
On Wed, Oct 17, 2012 at 12:14 PM, Lucas De Marchi
<lucas.demarchi@xxxxxxxxxxxxxx> wrote:
> @@ -827,24 +824,21 @@ static void set_name(struct btd_adapter *adapter, const char *name,
> int ret;
>
> if (adapter->allow_name_changes == FALSE)
> - return g_dbus_pending_property_error(btd_get_dbus_connection(),
> - id, ERROR_INTERFACE ".Failed",
> + return g_dbus_pending_property_error(id,
> + ERROR_INTERFACE ".Failed",
> strerror(EPERM));
>
> ret = adapter_set_name(adapter, name);
> - if (ret >= 0) {
> - g_dbus_pending_property_success(btd_get_dbus_connection(), id);
> - return;
> - }
> + if (ret >= 0)
> + return g_dbus_pending_property_success(id);
Out of curiosity, is this style of returning "void" (instead of just
"return;") common in gdbus?
Regards,
--
Anderson Lizardo
Instituto Nokia de Tecnologia - INdT
Manaus - Brazil
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
