Here is a rebased version of the patches. Most notable change is on patch
implementing the Set() method after feedback from Marcel. It doesn't cover the
concerns from Luiz about checking privileges per-property since I think this
could be added in a separate patch. As far as I could see the only user is
MediaTransport interface. The most obvious way would be to add another hook in
GDBusPropertyTable in order to check the privileges. Suggestions?
First 2 patches could be applied nonetheles. Get and GetAll are well tested,
both now and in the previous version of this patch set. Set() is still a
bit raw - the users implementing it are very big (adapter, device) so
they are not converted yet (previous patch doesn't apply anymore and there's a
change in the API that requires them to be rewritten).
Lucas De Marchi (10):
gdbus: Move typedefs up
gdbus: Use macros to add annotations
gdbus: Add skeleton of DBus.Properties interface
gdbus: Implement DBus.Properties.Get method
gdbus: Implement DBus.Properties.GetAll method
gdbus: Implement DBus.Properties.Set method
gdbus: Add properties into Introspectable interface
gdbus: Implement PropertiesChanged signal
Use DBus.Properties on Control interface
Use DBus.Properties on Manager interface
Luiz Augusto von Dentz (5):
gdbus: Add support for org.freedesktop.DBus.ObjectManager interface
gdbus: Group interface changes to reduce the amount of signals
emitted
gdbus: Only export ObjectManager interface on root path
gdbus: Integrates ObjectManager with Properties interface
gdbus: Simplify code for appending properties
audio/control.c | 57 ++--
gdbus/gdbus.h | 74 +++--
gdbus/object.c | 867 +++++++++++++++++++++++++++++++++++++++++++++++++++-----
src/manager.c | 81 ++----
4 files changed, 897 insertions(+), 182 deletions(-)
--
1.7.12.2
--
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
