[Tizen] Dali::Vector release memory later during Copy 53/268353/2 accepted/tizen/6.5/unified/20220104.122946 submit/tizen_6.5/20220103.151838
authorEunki, Hong <eunkiki.hong@samsung.com>
Tue, 21 Dec 2021 12:02:42 +0000 (21:02 +0900)
committerEunki, Hong <eunkiki.hong@samsung.com>
Fri, 31 Dec 2021 06:34:39 +0000 (15:34 +0900)
Change-Id: I16bcdca4737a9f754b22089873f4c42545ceae40
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
dali/public-api/common/dali-vector.cpp

index 80b03a4..3ab1396 100644 (file)
@@ -112,16 +112,19 @@ void VectorBase::Reserve(SizeType capacity, SizeType elementSize)
 
 void VectorBase::Copy(const VectorBase& vector, SizeType elementSize)
 {
-  // release old data
-  Release();
   // reserve space based on source capacity
-  const SizeType capacity = vector.Capacity();
-  Reserve(capacity, elementSize);
-  // copy over whole data
+  const SizeType capacity        = vector.Capacity();
   const SizeType wholeAllocation = sizeof(SizeType) * 2u + capacity * elementSize;
-  SizeType*      srcData         = reinterpret_cast<SizeType*>(vector.mData);
-  SizeType*      dstData         = reinterpret_cast<SizeType*>(mData);
+  void*          wholeData       = reinterpret_cast<void*>(new uint8_t[wholeAllocation]);
+  DALI_ASSERT_ALWAYS(wholeData && "VectorBase::Copy - Memory allocation failed");
+
+  // copy over whole data
+  SizeType* srcData = reinterpret_cast<SizeType*>(vector.mData);
+  SizeType* dstData = reinterpret_cast<SizeType*>(wholeData) + 2u;
   memcpy(dstData - 2u, srcData - 2u, wholeAllocation);
+
+  // release old buffer and set new data as mData
+  Replace(reinterpret_cast<void*>(dstData));
 }
 
 void VectorBase::Swap(VectorBase& vector)