NekoCWD/libcamera
NekoCWD/libcamera
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 7 Wiki Activity Actions

apps: qcam: Remove redundant check

Browse source 
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@protonmail.com>
Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Reviewed-by: Umang Jain <umang.jain@ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
This commit is contained in:
Barnabás Pőcze 2023-05-02 17:55:53 +00:00 • committed by Jacopo Mondi
parent e7d7c99f4b
commit 8258bd6ad6
1 changed files with 2 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/apps/qcam/main_window.cpp
View file

@ -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. */