[PATCH 01/10] libcamera: base: log: Declare log categories when defining them
Kieran Bingham
kieran.bingham at ideasonboard.com
Mon Jun 24 22:14:48 CEST 2024
Quoting Laurent Pinchart (2024-06-24 20:29:32)
> libcamera will enable -Wmissing-declarations to catch mismatches between
> function declarations and definitions. There is one offender in log.h:
> when a category is defined with LOG_DEFINE_CATEGORY(), it generates a
> function with no declaration. Fix it by declaring the function using
> LOG_DECLARE_CATEGORY() as the first step fo the category definition.
s/fo/of/
or
s/fo/for/
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
> include/libcamera/base/log.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/libcamera/base/log.h b/include/libcamera/base/log.h
> index b5775e491fee..df27df034d79 100644
> --- a/include/libcamera/base/log.h
> +++ b/include/libcamera/base/log.h
> @@ -48,6 +48,7 @@ private:
> extern const LogCategory &_LOG_CATEGORY(name)();
>
> #define LOG_DEFINE_CATEGORY(name) \
> +LOG_DECLARE_CATEGORY(name) \
> const LogCategory &_LOG_CATEGORY(name)() \
> { \
> /* The instance will be deleted by the Logger destructor. */ \
> --
> Regards,
>
> Laurent Pinchart
>
More information about the libcamera-devel
mailing list