All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] gprs: Fix the ContextAdded() signal Introspection
@ 2015-12-22  8:15 John Ernberg
  2015-12-22 16:24 ` Denis Kenzior
  0 siblings, 1 reply; 2+ messages in thread
From: John Ernberg @ 2015-12-22  8:15 UTC (permalink / raw)
  To: ofono

[-- Attachment #1: Type: text/plain, Size: 820 bytes --]

From: John Ernberg <john.ernberg@actia.se>

This matches the behavior described by the documentation the signal
value returned by the code. This was causing a headache when using
stricter D-Bus wrappers like dbus-c++.
---
 src/gprs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gprs.c b/src/gprs.c
index ba797dd..9192e68 100644
--- a/src/gprs.c
+++ b/src/gprs.c
@@ -2385,7 +2385,7 @@ static const GDBusSignalTable manager_signals[] = {
 	{ GDBUS_SIGNAL("PropertyChanged",
 			GDBUS_ARGS({ "name", "s" }, { "value", "v" })) },
 	{ GDBUS_SIGNAL("ContextAdded",
-			GDBUS_ARGS({ "path", "o" }, { "properties", "v" })) },
+			GDBUS_ARGS({ "path", "o" }, { "properties", "a{sv}" })) },
 	{ GDBUS_SIGNAL("ContextRemoved", GDBUS_ARGS({ "path", "o" })) },
 	{ }
 };
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] gprs: Fix the ContextAdded() signal Introspection
  2015-12-22  8:15 [PATCH] gprs: Fix the ContextAdded() signal Introspection John Ernberg
@ 2015-12-22 16:24 ` Denis Kenzior
  0 siblings, 0 replies; 2+ messages in thread
From: Denis Kenzior @ 2015-12-22 16:24 UTC (permalink / raw)
  To: ofono

[-- Attachment #1: Type: text/plain, Size: 415 bytes --]

Hi John,

On 12/22/2015 02:15 AM, John Ernberg wrote:
> From: John Ernberg <john.ernberg@actia.se>
>
> This matches the behavior described by the documentation the signal
> value returned by the code. This was causing a headache when using
> stricter D-Bus wrappers like dbus-c++.
> ---
>   src/gprs.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>

Applied, thanks.

Regards,
-Denis


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-12-22 16:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-22  8:15 [PATCH] gprs: Fix the ContextAdded() signal Introspection John Ernberg
2015-12-22 16:24 ` Denis Kenzior

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.