diff options
| author | Christian Rauch via libcamera-devel <libcamera-devel@lists.libcamera.org> | 2022-08-16 20:38:25 +0200 |
|---|---|---|
| committer | Jacopo Mondi <jacopo@jmondi.org> | 2022-08-25 14:03:22 +0200 |
| commit | 45c198da638d603c1a0cac216633933abfbb4153 (patch) | |
| tree | c58da0834d7e772961dd9e116f93b05ccebc8d86 /src/libcamera/pipeline/uvcvideo/uvcvideo.cpp | |
| parent | 868ab2287da2def1cc6414c27d9e26fd1547f353 (diff) | |
libcamera: Use const reference for range loops
Use a const reference in range-based for loops to avoid copies of the
loop elements.
While at it, change looping over controls in
PipelineHandlerUVC::processControls to use structured bindings.
Signed-off-by: Christian Rauch <Rauch.Christian@gmx.de>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'src/libcamera/pipeline/uvcvideo/uvcvideo.cpp')
| -rw-r--r-- | src/libcamera/pipeline/uvcvideo/uvcvideo.cpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp b/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp index fbe02cdc..9cbf126a 100644 --- a/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp +++ b/src/libcamera/pipeline/uvcvideo/uvcvideo.cpp @@ -340,12 +340,8 @@ int PipelineHandlerUVC::processControls(UVCCameraData *data, Request *request) { ControlList controls(data->video_->controls()); - for (auto it : request->controls()) { - unsigned int id = it.first; - ControlValue &value = it.second; - + for (const auto &[id, value] : request->controls()) processControl(&controls, id, value); - } for (const auto &ctrl : controls) LOG(UVC, Debug) |