Re: [PATCH] watchlist: added macro for notifying with no args

[James Prestwood <james.prestwood@linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1549 bytes --]

Sorry, did I not mean to send to this list.


On 11/14/2017 09:24 AM, James Prestwood wrote:
> The macros currently did not support notifying a watchlist with
> no extra arguments.
> ---
>   src/watchlist.h | 18 +++++++++++++++++-
>   1 file changed, 17 insertions(+), 1 deletion(-)
>
> diff --git a/src/watchlist.h b/src/watchlist.h
> index 737b80d..3d87810 100644
> --- a/src/watchlist.h
> +++ b/src/watchlist.h
> @@ -2,7 +2,7 @@
>    *
>    *  Wireless daemon for Linux
>    *
> - *  Copyright (C) 2016  Intel Corporation. All rights reserved.
> + *  Copyright (C) 2016-2017  Intel Corporation. All rights reserved.
>    *
>    *  This library is free software; you can redistribute it and/or
>    *  modify it under the terms of the GNU Lesser General Public
> @@ -92,3 +92,19 @@ void __watchlist_prune_stale(struct watchlist *watchlist);
>   		if ((watchlist)->stale_items)				\
>   			__watchlist_prune_stale(watchlist);		\
>   	} while	(false)
> +
> +#define WATCHLIST_NOTIFY_NO_ARGS(watchlist, type)			\
> +	do {								\
> +		const struct l_queue_entry *entry =			\
> +				l_queue_get_entries((watchlist)->items);\
> +									\
> +		(watchlist)->in_notify = true;				\
> +		for (; entry; entry = entry->next) {			\
> +			struct watchlist_item *item = entry->data;	\
> +			type t = item->notify;				\
> +			t(item->notify_data);			\
> +		}							\
> +		(watchlist)->in_notify = false;				\
> +		if ((watchlist)->stale_items)				\
> +			__watchlist_prune_stale(watchlist);		\
> +	} while	(false)