Hi Lucas,
> Contrary to Get() and GetAll(), Set is asynchronous so we pass on the
> DBusMessage so user is able to create the response. It's the only use of
> this parameter.
> ---
> gdbus/gdbus.h | 7 +++++++
> gdbus/object.c | 59 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++--
> 2 files changed, 64 insertions(+), 2 deletions(-)
>
> diff --git a/gdbus/gdbus.h b/gdbus/gdbus.h
> index b2e78c4..3e4aa16 100644
> --- a/gdbus/gdbus.h
> +++ b/gdbus/gdbus.h
> @@ -31,6 +31,8 @@ extern "C" {
> #include <dbus/dbus.h>
> #include <glib.h>
>
> +typedef enum GDBusPropertySetReturn GDBusPropertySetReturn;
> +
> typedef enum GDBusMethodFlags GDBusMethodFlags;
> typedef enum GDBusSignalFlags GDBusSignalFlags;
> typedef enum GDBusPropertyFlags GDBusPropertyFlags;
> @@ -69,6 +71,10 @@ typedef DBusMessage * (* GDBusMethodFunction) (DBusConnection *connection,
> typedef gboolean (*GDBusPropertyGetter)(const GDBusPropertyTable *property,
> DBusMessageIter *iter, void *data);
>
> +typedef DBusMessage *(*GDBusPropertySetter)(const GDBusPropertyTable *property,
> + DBusMessageIter *value,
> + DBusMessage *msg, void *data);
> +
I am not really happy with this. We just need a unique handle here since
the return value is either success or an error. I rather don't send
messages around for no other reason to create the error.
Inside the authorization code I am using GDBusPendingReply as unique
token. What about using the same approach?
Regards
Marcel
--
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
