Hi Michael,
> On Thu, Mar 12, 2015 at 10:11 AM, Michael Janssen <jamuraa@xxxxxxxxxxxx> wrote:
> Start using the newly introduced struct advertising_data API.
> ---
> src/advertising-manager.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/src/advertising-manager.c b/src/advertising-manager.c
> index 5045a06..3f53637 100644
> --- a/src/advertising-manager.c
> +++ b/src/advertising-manager.c
> @@ -32,6 +32,7 @@
> #include "dbus-common.h"
> #include "error.h"
> #include "log.h"
> +#include "src/shared/advertising-data.h"
> #include "src/shared/queue.h"
> #include "src/shared/util.h"
>
> @@ -56,6 +57,7 @@ struct advertisement {
> uint8_t type; /* Advertising type */
> bool random;
> bool published;
> + struct advertising_data *data;
> };
>
> static bool match_advertisement_path(const void *a, const void *b)
> @@ -76,6 +78,9 @@ static void advertisement_free(struct advertisement *ad)
> g_dbus_client_unref(ad->client);
> }
>
> + if (ad->data)
> + advertising_data_unref(ad->data);
> +
The checks for NULL are probably not needed since unref should
silently fail if NULL is passed.
> if (ad->proxy)
> g_dbus_proxy_unref(ad->proxy);
>
> @@ -272,6 +277,10 @@ static struct advertisement *advertisement_create(DBusConnection *conn,
> if (!ad->path)
> goto fail;
>
> + ad->data = advertising_data_new();
> + if (!ad->data)
> + goto fail;
> +
> ad->reg = dbus_message_ref(msg);
>
> g_dbus_client_set_disconnect_watch(ad->client, client_disconnect_cb,
> --
> 2.2.0.rc0.207.ga3a616c
>
> --
> 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
Arman
--
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
