tdf#163738 Use insert() to add multiple values in container instead of for loop

Signed-off-by: Stefan Dorneanu <cdmail2012.cd@gmail.com>
Change-Id: I6f34abdf199db2cfb14313a600652452ac328079
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/185432
Tested-by: Jenkins
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>

1 files changed, 3 insertions, 2 deletions

diff --git a/chart2/source/controller/dialogs/DialogModel.cxx b/chart2/source/controller/dialogs/DialogModel.cxx
index 45456d1327b8..1ea7ddfe56d1 100644
--- a/chart2/source/controller/dialogs/DialogModel.cxx
+++ b/chart2/source/controller/dialogs/DialogModel.cxx
@@ -402,9 +402,10 @@ std::vector< rtl::Reference< ChartType > >
                 xDiagram->getBaseCoordinateSystems());
             for( rtl::Reference< BaseCoordinateSystem > const & coords : aCooSysSeq )
             {
+                aResult.insert(aResult.end(),
+                               coords->getChartTypes2().begin(),
+                               coords->getChartTypes2().end());
 
-                for (const auto & rxChartType : coords->getChartTypes2())
-                    aResult.push_back(rxChartType);
             }
         }
     }