Use assign for deserialize instead of temporary std::string (#241)

Signed-off-by: Samuel Hyyppä <samuel.hyyppa@sandvik.com> Co-authored-by: Samuel Hyyppä <samuel.hyyppa@sandvik.com>
eProsima · Dec 17, 2024 · 5b8025b · 5b8025b
1 parent abf6c9a
commit 5b8025b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/include/fastcdr/Cdr.h b/include/fastcdr/Cdr.h
@@ -1793,7 +1793,7 @@ class Cdr
     {
         uint32_t length = 0;
         const char* str = read_string(length);
-        value = std::string(str, length);
+        value.assign(str, length);
         return *this;
     }
 

diff --git a/include/fastcdr/FastCdr.h b/include/fastcdr/FastCdr.h
@@ -1552,7 +1552,7 @@ class Cdr_DllAPI FastCdr
     {
         uint32_t length = 0;
         const char* str = read_string(length);
-        string_t = std::string(str, length);
+        string_t.assign(str, length);
         return *this;
     }