libcamera: camera_manager: Simplify camera lookup
`std::find()` works just fine because `std::shared_ptr` has `operator==()` with the expected semantics. Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
This commit is contained in:
@@ -239,10 +239,7 @@ void CameraManager::Private::removeCamera(std::shared_ptr<Camera> camera)
|
||||
{
|
||||
MutexLocker locker(mutex_);
|
||||
|
||||
auto iter = std::find_if(cameras_.begin(), cameras_.end(),
|
||||
[camera](std::shared_ptr<Camera> &c) {
|
||||
return c.get() == camera.get();
|
||||
});
|
||||
auto iter = std::find(cameras_.begin(), cameras_.end(), camera);
|
||||
if (iter == cameras_.end())
|
||||
return;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user