Re: [PATCH v2 0/6] MAP client: Add notification dispatching

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

 



Hi Christian,

On Mon, Jun 17, 2013 at 11:35 AM, Christian Fetzer
<christian.fetzer@xxxxxxxxxxxxxxxx> wrote:
> From: Christian Fetzer <christian.fetzer@xxxxxxxxxxxx>
>
> Changelog:
> v2:
>   - Implement dispatching in map-event.c
>   - Rename map_acquire_mns to map_register_event_handler
>   - Pass obc_session directly instead of copying address
> --
>
> This patch set completes the two patch sets "Add Message Notification Service
> (MNS) server" and "Read MASInstanceID for the current connection".
>
> The notification dispatcher component dispatches event reports that are
> received by the MNS server to the correct MAS client instance.
>
> The message access part of every client instance registers a notification
> handler function. The MNS will call the associated handler for all received
> event reports.
>
> For now, the handler will only print a log message for the received
> notifications. This is the starting point where we can add the necessary
> DBus API parts.
>
> Christian Fetzer (6):
>   obexd: Add function to get the destination from the obc_session
>   obexd: Add MAP notification dispatching
>   build: Add support for building MAP notification dispatcher
>   obexd: Register notification handler in the MAP client
>   obexd: Notify registered notification handlers in MNS
>   obexd: Fix crash when retrieving multiple MAP event reports
>
>  Makefile.obexd           |   1 +
>  obexd/client/map-event.c | 112 +++++++++++++++++++++++++++++++++++++++++++++++
>  obexd/client/map-event.h |  29 ++++++++++++
>  obexd/client/map.c       |  25 +++++++++++
>  obexd/client/mns.c       |  18 ++++----
>  obexd/client/session.c   |   5 +++
>  obexd/client/session.h   |   1 +
>  7 files changed, 181 insertions(+), 10 deletions(-)
>  create mode 100644 obexd/client/map-event.c
>
> --
> 1.8.2.3

Ive applied the rest of this set, note that I have do a lot of
cleanups I hope you can take a look because at first it was a bit
convoluted specially in map-event.c. I also found a very clear bug in
the first attempt to trigger a notification so I hope next time we can
do a smoke test at least.

--
Luiz Augusto von Dentz
--
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




[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