[PATCH v5 03/18] libcamera: Unify Doxygen file directive prefix for formats.h

Dan Scally dan.scally at ideasonboard.com
Tue Aug 6 15:48:32 CEST 2024


Hi Laurent

On 05/08/2024 15:36, Laurent Pinchart wrote:
> libcamera has two formats.h headers, an internal one in
> include/libcamera/internal/, and a public one generated at build time.
> The convention is to prefix the internal header name with
> libcamera/internal/ in the Doxygen file directive, but formats.cpp only
> uses internal/ as a prefix. Unify it with the rest of the code base.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---


Reviewed-by: Daniel Scally <dan.scally at ideasonboard.com>

>   src/libcamera/formats.cpp | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/libcamera/formats.cpp b/src/libcamera/formats.cpp
> index cf41f2c261ed..72fade7cd41c 100644
> --- a/src/libcamera/formats.cpp
> +++ b/src/libcamera/formats.cpp
> @@ -16,7 +16,7 @@
>   #include <libcamera/formats.h>
>   
>   /**
> - * \file internal/formats.h
> + * \file libcamera/internal/formats.h
>    * \brief Types and helper functions to handle libcamera image formats
>    */
>   


More information about the libcamera-devel mailing list