From de98dd0871a8c091b8413bd9172416a3afe54f71 Mon Sep 17 00:00:00 2001
From: "Robin.Mueller" <robin.mueller.m@gmail.com>
Date: Tue, 25 Aug 2020 13:37:42 +0200
Subject: [PATCH] optimization

---
 serialize/SerialBufferAdapter.cpp | 14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/serialize/SerialBufferAdapter.cpp b/serialize/SerialBufferAdapter.cpp
index c300eb36a..ab52262fb 100644
--- a/serialize/SerialBufferAdapter.cpp
+++ b/serialize/SerialBufferAdapter.cpp
@@ -23,16 +23,6 @@ SerialBufferAdapter<count_t>::~SerialBufferAdapter() {
 template<typename count_t>
 ReturnValue_t SerialBufferAdapter<count_t>::serialize(uint8_t** buffer,
 		size_t* size, size_t maxSize, Endianness streamEndianness) const {
-	count_t serializedLength = bufferLength;
-	if (serializeLength) {
-		serializedLength += SerializeAdapter::getSerializedSize(
-				&bufferLength);
-	}
-
-	if (*size + serializedLength > maxSize) {
-		return BUFFER_TOO_SHORT;
-	} 
-
 	if (serializeLength) {
 		ReturnValue_t result = SerializeAdapter::serialize(&bufferLength,
 				buffer, size, maxSize, streamEndianness);
@@ -41,6 +31,10 @@ ReturnValue_t SerialBufferAdapter<count_t>::serialize(uint8_t** buffer,
 		}
 	}
 
+	if (*size + bufferLength > maxSize) {
+		return BUFFER_TOO_SHORT;
+	}
+
 	if (this->constBuffer != nullptr) {
 		std::memcpy(*buffer, this->constBuffer, bufferLength);
 	}