2012-05-21 14:54:35

by Lucas De Marchi

[permalink] [raw]
Subject: [PATCH BlueZ] gdbus: add macro for methods marked as NOREPLY

---

This one is not used in BlueZ, but it was necessary for other projects using
gdbus. It's better to keep them in sync.

gdbus/gdbus.h | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/gdbus/gdbus.h b/gdbus/gdbus.h
index e2e160d..0a8a27c 100644
--- a/gdbus/gdbus.h
+++ b/gdbus/gdbus.h
@@ -144,6 +144,13 @@ typedef struct {
.function = _function, \
.flags = G_DBUS_METHOD_FLAG_ASYNC | G_DBUS_METHOD_FLAG_DEPRECATED

+#define GDBUS_NOREPLY_METHOD(_name, _in_args, _out_args, _function) \
+ .name = _name, \
+ .in_args = _in_args, \
+ .out_args = _out_args, \
+ .function = _function, \
+ .flags = G_DBUS_METHOD_FLAG_NOREPLY
+
#define GDBUS_SIGNAL(_name, _args) \
.name = _name, \
.args = _args
--
1.7.10.2



2012-05-23 22:24:35

by Lucas De Marchi

[permalink] [raw]
Subject: Re: [PATCH BlueZ] gdbus: add macro for methods marked as NOREPLY

Hi Marcel,

On Mon, May 21, 2012 at 11:54 AM, Lucas De Marchi
<[email protected]> wrote:
> ---
>
> This one is not used in BlueZ, but it was necessary for other projects us=
ing
> gdbus. It's better to keep them in sync.
>
> =A0gdbus/gdbus.h | =A0 =A07 +++++++
> =A01 file changed, 7 insertions(+)
>
> diff --git a/gdbus/gdbus.h b/gdbus/gdbus.h
> index e2e160d..0a8a27c 100644
> --- a/gdbus/gdbus.h
> +++ b/gdbus/gdbus.h

This one is still missing in BlueZ, neard and ConnMan.


Lucas De Marchi

2012-06-18 14:34:17

by Johan Hedberg

[permalink] [raw]
Subject: Re: [PATCH BlueZ] gdbus: add macro for methods marked as NOREPLY

Hi Lucas,

On Wed, May 23, 2012, Lucas De Marchi wrote:
> On Mon, May 21, 2012 at 11:54 AM, Lucas De Marchi
> <[email protected]> wrote:
> > ---
> >
> > This one is not used in BlueZ, but it was necessary for other projects using
> > gdbus. It's better to keep them in sync.
> >
> > ?gdbus/gdbus.h | ? ?7 +++++++
> > ?1 file changed, 7 insertions(+)
> >
> > diff --git a/gdbus/gdbus.h b/gdbus/gdbus.h
> > index e2e160d..0a8a27c 100644
> > --- a/gdbus/gdbus.h
> > +++ b/gdbus/gdbus.h
>
> This one is still missing in BlueZ, neard and ConnMan.

Applied to bluez.git and obexd.git.

Johan