[libcamera-devel] [PATCH] ipc: ipa_data_serializer: Use cend() in deserializers

[Paul Elder]

The overloaded deserializers functions that called the main deserializer
function using iterators used cbegin() but end() instead of cend(). Make
all of these use cend() instead.

Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
---
 include/libcamera/internal/ipa_data_serializer.h | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/include/libcamera/internal/ipa_data_serializer.h b/include/libcamera/internal/ipa_data_serializer.h
index 30bdaebc..54895b87 100644
--- a/include/libcamera/internal/ipa_data_serializer.h
+++ b/include/libcamera/internal/ipa_data_serializer.h
@@ -134,7 +134,7 @@ public:
 
 	static std::vector<V> deserialize(std::vector<uint8_t> &data, ControlSerializer *cs = nullptr)
 	{
-		return deserialize(data.cbegin(), data.end(), cs);
+		return deserialize(data.cbegin(), data.cend(), cs);
 	}
 
 	static std::vector<V> deserialize(std::vector<uint8_t>::const_iterator dataBegin,
@@ -142,13 +142,13 @@ public:
 					  ControlSerializer *cs = nullptr)
 	{
 		std::vector<SharedFD> fds;
-		return deserialize(dataBegin, dataEnd, fds.cbegin(), fds.end(), cs);
+		return deserialize(dataBegin, dataEnd, fds.cbegin(), fds.cend(), cs);
 	}
 
 	static std::vector<V> deserialize(std::vector<uint8_t> &data, std::vector<SharedFD> &fds,
 					  ControlSerializer *cs = nullptr)
 	{
-		return deserialize(data.cbegin(), data.end(), fds.cbegin(), fds.end(), cs);
+		return deserialize(data.cbegin(), data.cend(), fds.cbegin(), fds.cend(), cs);
 	}
 
 	static std::vector<V> deserialize(std::vector<uint8_t>::const_iterator dataBegin,
@@ -240,7 +240,7 @@ public:
 
 	static std::map<K, V> deserialize(std::vector<uint8_t> &data, ControlSerializer *cs = nullptr)
 	{
-		return deserialize(data.cbegin(), data.end(), cs);
+		return deserialize(data.cbegin(), data.cend(), cs);
 	}
 
 	static std::map<K, V> deserialize(std::vector<uint8_t>::const_iterator dataBegin,
@@ -248,13 +248,13 @@ public:
 					  ControlSerializer *cs = nullptr)
 	{
 		std::vector<SharedFD> fds;
-		return deserialize(dataBegin, dataEnd, fds.cbegin(), fds.end(), cs);
+		return deserialize(dataBegin, dataEnd, fds.cbegin(), fds.cend(), cs);
 	}
 
 	static std::map<K, V> deserialize(std::vector<uint8_t> &data, std::vector<SharedFD> &fds,
 					  ControlSerializer *cs = nullptr)
 	{
-		return deserialize(data.cbegin(), data.end(), fds.cbegin(), fds.end(), cs);
+		return deserialize(data.cbegin(), data.cend(), fds.cbegin(), fds.cend(), cs);
 	}
 
 	static std::map<K, V> deserialize(std::vector<uint8_t>::const_iterator dataBegin,
-- 
2.30.2