diff options
| author | Daniel Scally <dan.scally@ideasonboard.com> | 2025-02-20 22:05:32 +0000 |
|---|---|---|
| committer | Daniel Scally <dan.scally@ideasonboard.com> | 2025-05-29 11:56:23 +0100 |
| commit | f58077f073f74f3b8f325e0d75822f029b32c597 (patch) | |
| tree | 2e3ebdd2a40c2169b6e3974f76df77c42f12185b | |
| parent | b55943714f836658b9095503153a1c94bf17c916 (diff) | |
libcamera: mali-c55: Remove tpgSizes_ member from MaliC55CameraData
The tpgSizes_ vector is only used within the initTPGData() function.
Drop it and use a local variable instead.
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Daniel Scally <dan.scally@ideasonboard.com>
| -rw-r--r-- | src/libcamera/pipeline/mali-c55/mali-c55.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/libcamera/pipeline/mali-c55/mali-c55.cpp b/src/libcamera/pipeline/mali-c55/mali-c55.cpp index 35372ee1..f5260f5f 100644 --- a/src/libcamera/pipeline/mali-c55/mali-c55.cpp +++ b/src/libcamera/pipeline/mali-c55/mali-c55.cpp @@ -131,7 +131,6 @@ private: void setSensorControls(const ControlList &sensorControls); std::string id_; - std::vector<Size> tpgSizes_; Size tpgResolution_; }; @@ -179,13 +178,15 @@ void MaliC55CameraData::initTPGData() if (formats.empty()) return; + std::vector<Size> tpgSizes; + for (const auto &format : formats) { const std::vector<SizeRange> &ranges = format.second; - std::transform(ranges.begin(), ranges.end(), std::back_inserter(tpgSizes_), + std::transform(ranges.begin(), ranges.end(), std::back_inserter(tpgSizes), [](const SizeRange &range) { return range.max; }); } - tpgResolution_ = tpgSizes_.back(); + tpgResolution_ = tpgSizes.back(); } void MaliC55CameraData::setSensorControls(const ControlList &sensorControls) |