[PATCH v5 01/18] libcamera: Drop path prefix from Doxygen file directive
Kieran Bingham
kieran.bingham at ideasonboard.com
Wed Aug 7 06:45:20 CEST 2024
Quoting Laurent Pinchart (2024-08-05 15:36:37)
> The Doxygen directive only requires qualifying header file names with a
> path to differentiate between multiple header files with the same name.
> Most file directives that refer to unambiguous files do not have a
> libcamera/ and/or internal/ path prefix, but a few do, most likely due
> to copy&paste. Drop the prefix in those few files for consistency.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> ---
> src/libcamera/converter.cpp | 2 +-
> src/libcamera/converter/converter_v4l2_m2m.cpp | 2 +-
> src/libcamera/fence.cpp | 2 +-
> src/libcamera/mapped_framebuffer.cpp | 2 +-
> src/libcamera/orientation.cpp | 2 +-
> src/libcamera/yaml_parser.cpp | 2 +-
> 6 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/src/libcamera/converter.cpp b/src/libcamera/converter.cpp
> index 2ab461332677..8237998f044b 100644
> --- a/src/libcamera/converter.cpp
> +++ b/src/libcamera/converter.cpp
> @@ -14,7 +14,7 @@
> #include "libcamera/internal/media_device.h"
>
> /**
> - * \file internal/converter.h
> + * \file converter.h
> * \brief Abstract converter
> */
>
> diff --git a/src/libcamera/converter/converter_v4l2_m2m.cpp b/src/libcamera/converter/converter_v4l2_m2m.cpp
> index 2e77872e1710..006ba9f7f665 100644
> --- a/src/libcamera/converter/converter_v4l2_m2m.cpp
> +++ b/src/libcamera/converter/converter_v4l2_m2m.cpp
> @@ -23,7 +23,7 @@
> #include "libcamera/internal/v4l2_videodevice.h"
>
> /**
> - * \file internal/converter/converter_v4l2_m2m.h
> + * \file converter/converter_v4l2_m2m.h
> * \brief V4L2 M2M based converter
> */
>
> diff --git a/src/libcamera/fence.cpp b/src/libcamera/fence.cpp
> index 634c74f8e13e..73299b402702 100644
> --- a/src/libcamera/fence.cpp
> +++ b/src/libcamera/fence.cpp
> @@ -11,7 +11,7 @@ namespace libcamera {
>
> /**
> *
> - * \file libcamera/fence.h
> + * \file fence.h
> * \brief Definition of the Fence class
> */
>
> diff --git a/src/libcamera/mapped_framebuffer.cpp b/src/libcamera/mapped_framebuffer.cpp
> index b3104e0504d4..ad82e1f6e49c 100644
> --- a/src/libcamera/mapped_framebuffer.cpp
> +++ b/src/libcamera/mapped_framebuffer.cpp
> @@ -16,7 +16,7 @@
> #include <libcamera/base/log.h>
>
> /**
> - * \file libcamera/internal/mapped_framebuffer.h
> + * \file mapped_framebuffer.h
> * \brief Frame buffer memory mapping support
> */
>
> diff --git a/src/libcamera/orientation.cpp b/src/libcamera/orientation.cpp
> index 47fd6a32502b..fd191197fb8c 100644
> --- a/src/libcamera/orientation.cpp
> +++ b/src/libcamera/orientation.cpp
> @@ -11,7 +11,7 @@
> #include <string>
>
> /**
> - * \file libcamera/orientation.h
> + * \file orientation.h
> * \brief Image orientation definition
> */
>
> diff --git a/src/libcamera/yaml_parser.cpp b/src/libcamera/yaml_parser.cpp
> index 025006bcdcdd..8b6a403898be 100644
> --- a/src/libcamera/yaml_parser.cpp
> +++ b/src/libcamera/yaml_parser.cpp
> @@ -18,7 +18,7 @@
> #include <yaml.h>
>
> /**
> - * \file libcamera/internal/yaml_parser.h
> + * \file yaml_parser.h
> * \brief A YAML parser helper
> */
>
> --
> Regards,
>
> Laurent Pinchart
>
More information about the libcamera-devel
mailing list