RE: [PATCH v2 1/6] obexd: Add initial support for MAP conversations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luiz,

> -----Original Message-----
> From: linux-bluetooth-owner@xxxxxxxxxxxxxxx <linux-bluetooth-
> owner@xxxxxxxxxxxxxxx> On Behalf Of Luiz Augusto von Dentz
> Sent: Wednesday, February 12, 2020 12:58 AM
> To: Kishore, Ajay <ajay.kishore@xxxxxxxxx>
> Cc: linux-bluetooth@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v2 1/6] obexd: Add initial support for MAP conversations
> 
> Hi Ajay,
> 
> On Mon, Feb 10, 2020 at 11:32 PM Ajay Kishore <ajay.kishore@xxxxxxxxx>
> wrote:
> >
> > Changes made to add a new method for MAP conversation listing i.e
> > "ListConversations" to handle conversation listing object
> > "x-bt/MAP-convo-listing".
> > ---
> >  obexd/client/map.c | 67
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 67 insertions(+)
> >
> > diff --git a/obexd/client/map.c b/obexd/client/map.c index
> > 550c5af..adf62d9 100644
> > --- a/obexd/client/map.c
> > +++ b/obexd/client/map.c
> > @@ -1560,6 +1560,69 @@ static DBusMessage
> *map_list_messages(DBusConnection *connection,
> >         return get_message_listing(map, message, folder, apparam);  }
> >
> > +static GObexApparam *parse_conversation_filters(GObexApparam
> *apparam,
> > +
> > +DBusMessageIter *iter) {
> > +       DBusMessageIter array;
> > +
> > +       if (dbus_message_iter_get_arg_type(iter) != DBUS_TYPE_ARRAY) {
> > +               DBG("Not of type array");
> > +               return NULL;
> > +       }
> > +
> > +       dbus_message_iter_recurse(iter, &array);
> > +
> > +       while (dbus_message_iter_get_arg_type(&array) ==
> DBUS_TYPE_DICT_ENTRY) {
> > +               const char *key;
> > +               DBusMessageIter value, entry;
> > +
> > +               dbus_message_iter_recurse(&array, &entry);
> > +               dbus_message_iter_get_basic(&entry, &key);
> > +
> > +               dbus_message_iter_next(&entry);
> > +               dbus_message_iter_recurse(&entry, &value);
> > +
> > +               /* TODO: Parse conversation filters */
> > +
> > +               dbus_message_iter_next(&array);
> > +       }
> > +       return apparam;
> > +}
> > +
> > +static DBusMessage *map_list_conversations(DBusConnection *connection,
> > +                                               DBusMessage *message,
> > +                                               void *user_data) {
> > +       struct map_data *map = user_data;
> > +       const char *folder;
> > +       GObexApparam *apparam;
> > +       DBusMessageIter args;
> > +
> > +       dbus_message_iter_init(message, &args);
> > +
> > +       if (dbus_message_iter_get_arg_type(&args) != DBUS_TYPE_STRING)
> > +               return g_dbus_create_error(message,
> > +                       ERROR_INTERFACE ".InvalidArguments", NULL);
> > +
> > +       dbus_message_iter_get_basic(&args, &folder);
> > +
> > +       apparam = g_obex_apparam_set_uint16(NULL,
> MAP_AP_MAXLISTCOUNT,
> > +                                                       DEFAULT_COUNT);
> > +       apparam = g_obex_apparam_set_uint16(apparam,
> MAP_AP_STARTOFFSET,
> > +
> > + DEFAULT_OFFSET);
> > +
> > +       dbus_message_iter_next(&args);
> > +
> > +       if (parse_conversation_filters(apparam, &args) == NULL) {
> > +               g_obex_apparam_free(apparam);
> > +               return g_dbus_create_error(message,
> > +                       ERROR_INTERFACE ".InvalidArguments", NULL);
> > +       }
> > +
> > +       /*TODO: Return conversation listing */
> > +       return NULL;
> > +}
> > +
> >  static char **get_filter_strs(uint64_t filter, int *size)  {
> >         char **list, **item;
> > @@ -1817,6 +1880,10 @@ static const GDBusMethodTable map_methods[] =
> {
> >                         GDBUS_ARGS({ "folder", "s" }, { "filter", "a{sv}" }),
> >                         GDBUS_ARGS({ "messages", "a{oa{sv}}" }),
> >                         map_list_messages) },
> > +       { GDBUS_ASYNC_METHOD("ListConversations",
> > +                       GDBUS_ARGS({ "folder", "s" }, { "filter", "a{sv}" }),
> > +                       GDBUS_ARGS({ "conversations", "a{oa{sv}}" }),
> > +                       map_list_conversations) },
> >         { GDBUS_METHOD("ListFilterFields",
> >                         NULL,
> >                         GDBUS_ARGS({ "fields", "as" }),
> > --
> > 2.7.4
> 
> I see some mix of v2 and v3 patches, I recommend updating the whole set to v3
> to make easier to review.
Done.
> 
> 
> --
> Luiz Augusto von Dentz

Thanks.
Ajay




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux