All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] voicecall: fix CreateMultiparty method introspection
@ 2016-04-22 13:06 John Ernberg
  2016-04-22 13:06 ` [PATCH] voicecall: limit g_drivers variable exposure John Ernberg
  2016-04-22 19:18 ` [PATCH] voicecall: fix CreateMultiparty method introspection Denis Kenzior
  0 siblings, 2 replies; 4+ messages in thread
From: John Ernberg @ 2016-04-22 13:06 UTC (permalink / raw)
  To: ofono

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

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

This caused problems with stricter dbus wrappers such as dbus-c++.

---
 src/voicecall.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/voicecall.c b/src/voicecall.c
index 2d15d12..bdad125 100644
--- a/src/voicecall.c
+++ b/src/voicecall.c
@@ -2165,7 +2165,7 @@ static const GDBusMethodTable manager_methods[] = {
 						GDBUS_ARGS({ "calls", "ao" }),
 						multiparty_private_chat) },
 	{ GDBUS_ASYNC_METHOD("CreateMultiparty",
-					NULL, GDBUS_ARGS({ "calls", "o" }),
+					NULL, GDBUS_ARGS({ "calls", "ao" }),
 					multiparty_create) },
 	{ GDBUS_ASYNC_METHOD("HangupMultiparty", NULL, NULL,
 							multiparty_hangup) },
-- 
1.9.1

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

* [PATCH] voicecall: limit g_drivers variable exposure
  2016-04-22 13:06 [PATCH] voicecall: fix CreateMultiparty method introspection John Ernberg
@ 2016-04-22 13:06 ` John Ernberg
  2016-04-22 19:19   ` Denis Kenzior
  2016-04-22 19:18 ` [PATCH] voicecall: fix CreateMultiparty method introspection Denis Kenzior
  1 sibling, 1 reply; 4+ messages in thread
From: John Ernberg @ 2016-04-22 13:06 UTC (permalink / raw)
  To: ofono

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

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

---
 src/voicecall.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/voicecall.c b/src/voicecall.c
index c9b1b43..2d15d12 100644
--- a/src/voicecall.c
+++ b/src/voicecall.c
@@ -47,7 +47,7 @@
 #define SETTINGS_STORE "voicecall"
 #define SETTINGS_GROUP "Settings"
 
-GSList *g_drivers = NULL;
+static GSList *g_drivers = NULL;
 
 struct ofono_voicecall {
 	GSList *call_list;
-- 
1.9.1

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

* Re: [PATCH] voicecall: fix CreateMultiparty method introspection
  2016-04-22 13:06 [PATCH] voicecall: fix CreateMultiparty method introspection John Ernberg
  2016-04-22 13:06 ` [PATCH] voicecall: limit g_drivers variable exposure John Ernberg
@ 2016-04-22 19:18 ` Denis Kenzior
  1 sibling, 0 replies; 4+ messages in thread
From: Denis Kenzior @ 2016-04-22 19:18 UTC (permalink / raw)
  To: ofono

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

Hi John,

On 04/22/2016 08:06 AM, John Ernberg wrote:
> From: John Ernberg <john.ernberg@actia.se>
>
> This caused problems with stricter dbus wrappers such as dbus-c++.
>
> ---
>   src/voicecall.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>

Applied, thanks.

Regards,
-Denis


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

* Re: [PATCH] voicecall: limit g_drivers variable exposure
  2016-04-22 13:06 ` [PATCH] voicecall: limit g_drivers variable exposure John Ernberg
@ 2016-04-22 19:19   ` Denis Kenzior
  0 siblings, 0 replies; 4+ messages in thread
From: Denis Kenzior @ 2016-04-22 19:19 UTC (permalink / raw)
  To: ofono

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

Hi John,

On 04/22/2016 08:06 AM, John Ernberg wrote:
> From: John Ernberg <john.ernberg@actia.se>
>
> ---
>   src/voicecall.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>

Applied, thanks.

Regards,
-Denis


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

end of thread, other threads:[~2016-04-22 19:19 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-22 13:06 [PATCH] voicecall: fix CreateMultiparty method introspection John Ernberg
2016-04-22 13:06 ` [PATCH] voicecall: limit g_drivers variable exposure John Ernberg
2016-04-22 19:19   ` Denis Kenzior
2016-04-22 19:18 ` [PATCH] voicecall: fix CreateMultiparty method introspection 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.