[libcamera-devel] [PATCH v2 05/11] libcamera: pipeline: Use existing variable definitions

[Laurent Pinchart]

Hi Kieran,

Thank you for the patch.

On Thu, Oct 15, 2020 at 11:37:44PM +0100, Kieran Bingham wrote:
> Prevent variable aliasing by removing the redeclaration of variables
> with the same name (and type) where the existing variable can be reused.
> 
> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  src/libcamera/pipeline/raspberrypi/raspberrypi.cpp | 2 +-
>  src/libcamera/pipeline/simple/simple.cpp           | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> index 85e0a1f26ab6..2d70d984a276 100644
> --- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> +++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> @@ -1008,7 +1008,7 @@ int PipelineHandlerRPi::queueAllBuffers(Camera *camera)
>  			 */
>  			unsigned int i;
>  			for (i = 0; i < data->dropFrameCount_; i++) {
> -				int ret = stream->queueBuffer(nullptr);
> +				ret = stream->queueBuffer(nullptr);
>  				if (ret)
>  					return ret;
>  			}
> diff --git a/src/libcamera/pipeline/simple/simple.cpp b/src/libcamera/pipeline/simple/simple.cpp
> index 999c44515023..33daa2fb1b7b 100644
> --- a/src/libcamera/pipeline/simple/simple.cpp
> +++ b/src/libcamera/pipeline/simple/simple.cpp
> @@ -612,8 +612,8 @@ int SimplePipelineHandler::configure(Camera *camera, CameraConfiguration *c)
>  	useConverter_ = config->needConversion();
>  
>  	if (useConverter_) {
> -		int ret = converter_->configure(pipeConfig.pixelFormat,
> -						pipeConfig.captureSize, &cfg);
> +		ret = converter_->configure(pipeConfig.pixelFormat,
> +					    pipeConfig.captureSize, &cfg);
>  		if (ret < 0) {
>  			LOG(SimplePipeline, Error)
>  				<< "Unable to configure converter";

-- 
Regards,

Laurent Pinchart