[libcamera-devel] [PATCH v1] qcam: Remove redundant check

[Umang Jain]

Hi,

On 5/2/23 11:25 PM, Barnabás Pőcze via libcamera-devel wrote:
> The switch statement checks `roles.size()` with cases for 1 and 2,
> so in the `default` branch, `role.size() > 2`, i.e. it is always
> different from 1, so the check is unnecessary.
>
> Signed-off-by: Barnabás Pőcze <pobrn at protonmail.com>

Reviewed-by: Umang Jain <umang.jain at ideasonboard.com>

> ---
>   src/apps/qcam/main_window.cpp | 7 ++-----
>   1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/src/apps/qcam/main_window.cpp b/src/apps/qcam/main_window.cpp
> index fb2db4aa..680668df 100644
> --- a/src/apps/qcam/main_window.cpp
> +++ b/src/apps/qcam/main_window.cpp
> @@ -381,11 +381,8 @@ int MainWindow::startCapture()
>   		}
>   		break;
>   	default:
> -		if (roles.size() != 1) {
> -			qWarning() << "Unsupported stream configuration";
> -			return -EINVAL;
> -		}
> -		break;
> +		qWarning() << "Unsupported stream configuration";
> +		return -EINVAL;
>   	}
>
>   	/* Configure the camera. */
> --
> 2.40.1
>
>