use std::vector::insert instead of push_back

because it will pre-allocate space and often is optimised to memcpy

Change-Id: I03ed7915f2762d3d27e378638052a47a28bbf096
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123588
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 1 insertions, 2 deletions

diff --git a/forms/source/component/ComboBox.cxx b/forms/source/component/ComboBox.cxx
index 9b1bd9b9c7bd..fe3260070d68 100644
--- a/forms/source/component/ComboBox.cxx
+++ b/forms/source/component/ComboBox.cxx
@@ -632,8 +632,7 @@ void OComboBoxModel::loadData( bool _bForce )
                 if (xFieldNames.is())
                 {
                     const Sequence<OUString> aFieldNames = xFieldNames->getElementNames();
-                    for (const OUString& rustrNames : aFieldNames)
-                        aStringList.push_back(rustrNames);
+                    aStringList.insert(aStringList.end(), aFieldNames.begin(), aFieldNames.end());
                 }
             }
             break;