From 79be15a5f1528229a40504a2592056e9dd6fd3c8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Barnab=C3=A1s=20P=C5=91cze?=
 <barnabas.pocze@ideasonboard.com>
Date: Mon, 3 Nov 2025 14:55:53 +0100
Subject: [PATCH] ipa: libipa: agc_mean_luminance: Avoid unnecessary copies
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

`constraintModes()` and `exposureModeHelpers()` need not return copies of
the internal objects, exposing const references is sufficient, so do that.

Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Reviewed-by: Daniel Scally <dan.scally@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
 src/ipa/libipa/agc_mean_luminance.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/ipa/libipa/agc_mean_luminance.h b/src/ipa/libipa/agc_mean_luminance.h
index d7ec548e..e5f164c3 100644
--- a/src/ipa/libipa/agc_mean_luminance.h
+++ b/src/ipa/libipa/agc_mean_luminance.h
@@ -53,12 +53,12 @@ public:
 	void setLimits(utils::Duration minExposureTime, utils::Duration maxExposureTime,
 		       double minGain, double maxGain, std::vector<AgcConstraint> constraints);
 
-	std::map<int32_t, std::vector<AgcConstraint>> constraintModes()
+	const std::map<int32_t, std::vector<AgcConstraint>> &constraintModes() const
 	{
 		return constraintModes_;
 	}
 
-	std::map<int32_t, std::shared_ptr<ExposureModeHelper>> exposureModeHelpers()
+	const std::map<int32_t, std::shared_ptr<ExposureModeHelper>> &exposureModeHelpers() const
 	{
 		return exposureModeHelpers_;
 	}