[libcamera-devel] [PATCH/RFC 3/3] v4l2: Replace manual loop counters with utils::enumerate()

[Niklas Söderlund]

Hi Laurent,

Thanks for your patch.

On 2021-04-23 05:09:32 +0300, Laurent Pinchart wrote:
> Use the newly introduced utils::enumerate() to replace manual loop
> counters. A local variable is needed, as utils::enumerate() requires an
> lvalue reference.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

I would prefers the 1/3 API but for the change itself,

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/v4l2/v4l2_compat_manager.cpp | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/src/v4l2/v4l2_compat_manager.cpp b/src/v4l2/v4l2_compat_manager.cpp
> index 90c0f0121a32..bf5bf8e3223d 100644
> --- a/src/v4l2/v4l2_compat_manager.cpp
> +++ b/src/v4l2/v4l2_compat_manager.cpp
> @@ -23,6 +23,7 @@
>  #include <libcamera/camera_manager.h>
>  
>  #include "libcamera/internal/log.h"
> +#include "libcamera/internal/utils.h"
>  
>  #include "v4l2_camera_file.h"
>  
> @@ -81,11 +82,11 @@ int V4L2CompatManager::start()
>  	 * For each Camera registered in the system, a V4L2CameraProxy gets
>  	 * created here to wrap a camera device.
>  	 */
> -	unsigned int index = 0;
> -	for (auto &camera : cm_->cameras()) {
> -		V4L2CameraProxy *proxy = new V4L2CameraProxy(index, camera);
> +	auto cameras = cm_->cameras();
> +	for (const auto camera : utils::enumerate(cameras)) {
> +		V4L2CameraProxy *proxy = new V4L2CameraProxy(camera.index,
> +							     camera.value);
>  		proxies_.emplace_back(proxy);
> -		++index;
>  	}
>  
>  	return 0;
> @@ -117,11 +118,10 @@ int V4L2CompatManager::getCameraIndex(int fd)
>  	if (!target)
>  		return -1;
>  
> -	unsigned int index = 0;
> -	for (auto &camera : cm_->cameras()) {
> -		if (camera == target)
> -			return index;
> -		++index;
> +	auto cameras = cm_->cameras();
> +	for (const auto camera : utils::enumerate(cameras)) {
> +		if (camera.value == target)
> +			return camera.index;
>  	}
>  
>  	return -1;
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund