[libcamera-devel] [PATCH] libcamera: log: Hide value of the ASSERT() macro

Umang Jain umang.jain at ideasonboard.com
Sat May 22 12:59:17 CEST 2021


Hi Laurent,

Thank you for the patch

On 5/22/21 5:53 AM, Laurent Pinchart wrote:
> ASSERT() is a function-like macro that expands to internal code. Hide
> the expansion from the generated documentation, as it would only confuse
> the reader.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Umang Jain <umang.jain at ideasonboard.com>
> ---
>   src/libcamera/log.cpp | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/libcamera/log.cpp b/src/libcamera/log.cpp
> index 985952249289..dd991647b9bc 100644
> --- a/src/libcamera/log.cpp
> +++ b/src/libcamera/log.cpp
> @@ -990,6 +990,7 @@ LogMessage _log(const LogCategory *category, LogSeverity severity,
>   
>   /**
>    * \def ASSERT(condition)
> + * \hideinitializer
>    * \brief Abort program execution if assertion fails
>    *
>    * If \a condition is false, ASSERT() logs an error message with the Fatal log



More information about the libcamera-devel mailing list