use push_back

This is a vector. No need for iterators.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

diff --git a/src/KaxBlock.cpp b/src/KaxBlock.cpp
index 90fb4f3d67e21acd3b40f44129e50fe83bc91177..041ba733657c20f059fdfddafa8e011a02ccb586 100644 (file)
--- a/src/KaxBlock.cpp
+++ b/src/KaxBlock.cpp
@@ -76,7 +76,6 @@ KaxInternalBlock::~KaxInternalBlock()
 
 KaxInternalBlock::KaxInternalBlock(const KaxInternalBlock & ElementToClone)
   :EbmlBinary(ElementToClone)
-  ,myBuffers(ElementToClone.myBuffers.size())
   ,Timecode(ElementToClone.Timecode)
   ,LocalTimecode(ElementToClone.LocalTimecode)
   ,bLocalTimecodeUsed(ElementToClone.bLocalTimecodeUsed)
@@ -84,11 +83,9 @@ KaxInternalBlock::KaxInternalBlock(const KaxInternalBlock & ElementToClone)
   ,ParentCluster(ElementToClone.ParentCluster) ///< \todo not exactly
 {
   // add a clone of the list
-  auto myItr = myBuffers.begin();
-  for (const auto& buffer : ElementToClone.myBuffers) {
-    *myItr = buffer->Clone();
-    ++myItr;
-  }
+  myBuffers.reserve(ElementToClone.myBuffers.size());
+  for (const auto& buffer : ElementToClone.myBuffers)
+    myBuffers.push_back(buffer->Clone());
 }