From bb4d48fbf9d0ad6be570d9c9b052f835f3a8b0d5 Mon Sep 17 00:00:00 2001 From: Tomi Valkeinen Date: Sat, 13 Sep 2025 18:20:35 +0300 Subject: fix: Use const references to avoid unnecessary variable copying This commit addresses cppcheck warnings about variables that can be declared as const references (constVariableReference). These variables were being copied unnecessarily when they could be accessed as const references, improving performance and code clarity. Fixed files: - kms++util/src/cpuframebuffer.cpp: Make plane reference const in destructor - kms++util/src/testpat.cpp: Make info and lambda parameter const references - utils/kmstest.cpp: Make loop variable const reference in output parsing --- kms++util/src/cpuframebuffer.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'kms++util/src/cpuframebuffer.cpp') diff --git a/kms++util/src/cpuframebuffer.cpp b/kms++util/src/cpuframebuffer.cpp index 8eac8c4..64ad531 100644 --- a/kms++util/src/cpuframebuffer.cpp +++ b/kms++util/src/cpuframebuffer.cpp @@ -26,7 +26,7 @@ CPUFramebuffer::CPUFramebuffer(uint32_t width, uint32_t height, PixelFormat form CPUFramebuffer::~CPUFramebuffer() { for (unsigned i = 0; i < m_num_planes; ++i) { - FramebufferPlane& plane = m_planes[i]; + const FramebufferPlane& plane = m_planes[i]; delete[] plane.map; } -- cgit v1.2.3