Open Source Telephony
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox