diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2017-07-25 11:58:57 +0200 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2017-07-25 12:52:11 +0200 |
commit | 0e1e33f053c235dd8b63c60f68280f9cf9571281 (patch) | |
tree | bfd88e94dd1b09e346343604aee4f206091a75b7 | |
parent | 90bbbac25433e09a690c074ec135654c0cc7b4b1 (diff) |
OOXMLPropertySet::add: resize->reserve + range-based loop
Change-Id: I22fd6893137fcc8815fa222fdfa59f7076887399
Reviewed-on: https://gerrit.libreoffice.org/40393
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
-rw-r--r-- | writerfilter/source/ooxml/OOXMLPropertySet.cxx | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/writerfilter/source/ooxml/OOXMLPropertySet.cxx b/writerfilter/source/ooxml/OOXMLPropertySet.cxx index da8a3dd1bf23..118850144686 100644 --- a/writerfilter/source/ooxml/OOXMLPropertySet.cxx +++ b/writerfilter/source/ooxml/OOXMLPropertySet.cxx @@ -399,10 +399,9 @@ void OOXMLPropertySet::add(const OOXMLPropertySet::Pointer_t& pPropertySet) if (pSet != nullptr) { - mProperties.resize(mProperties.size() + pSet->mProperties.size()); - for (OOXMLProperties_t::iterator aIt = pSet->mProperties.begin(); - aIt != pSet->mProperties.end(); ++aIt) - add(*aIt); + mProperties.reserve(mProperties.size() + pSet->mProperties.size()); + for (const auto& aIt: pSet->mProperties) + add(aIt); } } |