PawletOS/external_libcamera
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

software_isp: benchmark: Add missing _ postfix to measure data member

Browse Source 
All class data members should have a _ postifx, add the missing _ postfix
to the Benchmark::measure_ data member.

Signed-off-by: Hans de Goede <johannes.goede@oss.qualcomm.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Milan Zamazal <mzamazal@redhat.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
This commit is contained in:
Hans de Goede
2026-02-15 10:44:17 +01:00
committed by Kieran Bingham
parent 1dcf9957a4
commit f0a79dd68b
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
include/libcamera/internal/software_isp/benchmark.h
View File

@@ -28,7 +28,7 @@ public:
private:
timespec frameStartTime_;
bool measure;
bool measure_;
/* Skip 30 frames for things to stabilize then measure 30 frames */
unsigned int encounteredFrames_ = 0;
int64_t frameProcessTime_ = 0;

10
src/libcamera/software_isp/benchmark.cpp
View File

@@ -54,11 +54,11 @@ static inline int64_t timeDiff(timespec &after, timespec &before)
*/
void Benchmark::startFrame(void)
{
measure = framesToMeasure_ > 0 &&
encounteredFrames_ < skipBeforeMeasure_ + framesToMeasure_ &&
++encounteredFrames_ > skipBeforeMeasure_;
measure_ = framesToMeasure_ > 0 &&
encounteredFrames_ < skipBeforeMeasure_ + framesToMeasure_ &&
++encounteredFrames_ > skipBeforeMeasure_;
if (measure) {
if (measure_) {
frameStartTime_ = {};
clock_gettime(CLOCK_MONOTONIC_RAW, &frameStartTime_);
}
@@ -75,7 +75,7 @@ void Benchmark::startFrame(void)
*/
void Benchmark::finishFrame(void)
{
if (measure) {
if (measure_) {
timespec frameEndTime = {};
clock_gettime(CLOCK_MONOTONIC_RAW, &frameEndTime);
frameProcessTime_ += timeDiff(frameEndTime, frameStartTime_);