2022-03-23 11:56:24

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [PATCH BlueZ v2 1/9] log: Introduce DBG_IS_ENABLED

Hi Luiz,

> This introduces DBG_IS_ENABLE macro which can be used to check if
> BTD_DEBUG_FLAG_PRINT has been enabled for the current file.
> ---
> src/log.c | 12 ++++++++++++
> src/log.h | 11 +++++++++++
> 2 files changed, 23 insertions(+)
>
> diff --git a/src/log.c b/src/log.c
> index 0155a6bba..1157859ef 100644
> --- a/src/log.c
> +++ b/src/log.c
> @@ -179,6 +179,18 @@ void __btd_log_init(const char *debug, int detach)
> info("Bluetooth daemon %s", VERSION);
> }
>
> +bool __btd_log_is_enabled(const char *file)
> +{
> + struct btd_debug_desc *desc;
> +
> + for (desc = __start___debug; desc < __stop___debug; desc++) {
> + if (desc->file && g_pattern_match_simple(file, desc->file))
> + return desc->flags & BTD_DEBUG_FLAG_PRINT;
> + }
> +
> + return false;
> +}
> +

this is an expensive operation. What do you need this for?

Regards

Marcel