[libcamera-devel] [PATCH 1/2] android: camera_hal_config: Move include <filesystem> to .cpp file

[Hirokazu Honda]

Hi Laurent, thank you for the patch.

On Wed, May 26, 2021 at 9:09 AM Laurent Pinchart <
laurent.pinchart at ideasonboard.com> wrote:

> The <filesystem> header isn't needed in camera_hal_config.h, move its
> inclusion to camera_hal_config.cpp.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>

Reviewed-by: Hirokazu Honda <hiroh at chromium.org>


> ---
>  src/android/camera_hal_config.cpp | 1 +
>  src/android/camera_hal_config.h   | 1 -
>  2 files changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/android/camera_hal_config.cpp
> b/src/android/camera_hal_config.cpp
> index 12d69a3f5f3c..d15df2e30c2c 100644
> --- a/src/android/camera_hal_config.cpp
> +++ b/src/android/camera_hal_config.cpp
> @@ -6,6 +6,7 @@
>   */
>  #include "camera_hal_config.h"
>
> +#include <filesystem>
>  #include <stdio.h>
>  #include <stdlib.h>
>  #include <string>
> diff --git a/src/android/camera_hal_config.h
> b/src/android/camera_hal_config.h
> index 0555c6da2c20..97dc69c1def5 100644
> --- a/src/android/camera_hal_config.h
> +++ b/src/android/camera_hal_config.h
> @@ -7,7 +7,6 @@
>  #ifndef __ANDROID_CAMERA_HAL_CONFIG_H__
>  #define __ANDROID_CAMERA_HAL_CONFIG_H__
>
> -#include <filesystem>
>  #include <map>
>  #include <string>
>
> --
> Regards,
>
> Laurent Pinchart
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.libcamera.org/pipermail/libcamera-devel/attachments/20210526/a2a0a468/attachment.htm>