Use std::vector::reserve in Element::Clone

There are several for-loops in Element::Clone that call
std::vector::push_back, and since the vector size is
known in advance, use std::vector::reserve to avoid
multiple vector resizes.

Signed-off-by: Steve Peters <[email protected]>

src/Element.cc

@@ -252,6 +252,7 @@ ElementPtr Element::Clone(sdf::Errors &_errors) const
   clone->dataPtr->explicitlySetInFile = this->dataPtr->explicitlySetInFile;
 
   Param_V::const_iterator aiter;
+  clone->dataPtr->attributes.reserve(this->dataPtr->attributes.size());
   for (aiter = this->dataPtr->attributes.begin();
        aiter != this->dataPtr->attributes.end(); ++aiter)
   {
@@ -263,12 +264,15 @@ ElementPtr Element::Clone(sdf::Errors &_errors) const
   }
 
   ElementPtr_V::const_iterator eiter;
+  clone->dataPtr->elementDescriptions.reserve(
+    this->dataPtr->elementDescriptions.size());
   for (eiter = this->dataPtr->elementDescriptions.begin();
       eiter != this->dataPtr->elementDescriptions.end(); ++eiter)
   {
     clone->dataPtr->elementDescriptions.push_back((*eiter)->Clone(_errors));
   }
 
+  clone->dataPtr->elements.reserve(this->dataPtr->elements.size());
   for (eiter = this->dataPtr->elements.begin();
        eiter != this->dataPtr->elements.end(); ++eiter)
   {