tdf#69977: uno::Sequence is expensive

when used as a mutable data-structure. Plain std::vector halves the time
taken to display the chart dialog

Create a class to represent the std::vector we are going to be passing
around, and move some of the utility methods into it to make the code
prettier.
Also create an optimised append(&&) method for the common case of
appending small temporaries.

Change-Id: I7f5b43fb4a8a84e40e6a52fcb7e9f974091b4485

1 files changed, 4 insertions, 2 deletions

diff --git a/svgio/qa/cppunit/SvgImportTest.cxx b/svgio/qa/cppunit/SvgImportTest.cxx
index 4b87b501a274..3c40e3e21df5 100644
--- a/svgio/qa/cppunit/SvgImportTest.cxx
+++ b/svgio/qa/cppunit/SvgImportTest.cxx
@@ -15,6 +15,7 @@
 
 #include <comphelper/processfactory.hxx>
 #include <comphelper/seqstream.hxx>
+#include <comphelper/sequence.hxx>
 
 #include <com/sun/star/graphic/SvgTools.hpp>
 #include <com/sun/star/graphic/Primitive2DTools.hpp>
@@ -32,6 +33,7 @@ using namespace css::io;
 using namespace css::graphic;
 using drawinglayer::primitive2d::arePrimitive2DSequencesEqual;
 using drawinglayer::primitive2d::Primitive2DSequence;
+using drawinglayer::primitive2d::Primitive2DContainer;
 
 class Test : public test::BootstrapFixture, public XmlTestTools
 {
@@ -84,7 +86,7 @@ void Test::tearDown()
 void Test::checkRectPrimitive(Primitive2DSequence& rPrimitive)
 {
     Primitive2dXmlDump dumper;
-    xmlDocPtr pDocument = dumper.dumpAndParse(rPrimitive);
+    xmlDocPtr pDocument = dumper.dumpAndParse(comphelper::sequenceToContainer<Primitive2DContainer>(rPrimitive));
 
     CPPUNIT_ASSERT (pDocument);
 
@@ -126,7 +128,7 @@ void Test::testTdf87309()
     CPPUNIT_ASSERT_EQUAL(1, (int)aSequenceTdf87309.getLength());
 
     Primitive2dXmlDump dumper;
-    xmlDocPtr pDocument = dumper.dumpAndParse(aSequenceTdf87309);
+    xmlDocPtr pDocument = dumper.dumpAndParse(comphelper::sequenceToContainer<Primitive2DContainer>(aSequenceTdf87309));
 
     CPPUNIT_ASSERT (pDocument);