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, 3 insertions, 3 deletions

diff --git a/test/source/primitive2dxmldump.cxx b/test/source/primitive2dxmldump.cxx
index 3d2328f51f36..c00f6e21b9b4 100644
--- a/test/source/primitive2dxmldump.cxx
+++ b/test/source/primitive2dxmldump.cxx
@@ -60,7 +60,7 @@ void Primitive2dXmlDump::filterAllActionTypes()
 }
 
 xmlDocPtr Primitive2dXmlDump::dumpAndParse(
-    const drawinglayer::primitive2d::Primitive2DSequence& rPrimitive2DSequence,
+    const drawinglayer::primitive2d::Primitive2DContainer& rPrimitive2DSequence,
     const OUString& rTempStreamName)
 {
     std::unique_ptr<SvStream> pStream;
@@ -87,10 +87,10 @@ xmlDocPtr Primitive2dXmlDump::dumpAndParse(
 }
 
 void Primitive2dXmlDump::decomposeAndWrite(
-    const drawinglayer::primitive2d::Primitive2DSequence& rPrimitive2DSequence,
+    const drawinglayer::primitive2d::Primitive2DContainer& rPrimitive2DSequence,
     XmlWriter& rWriter)
 {
-    for (int i = 0; i < rPrimitive2DSequence.getLength(); i++)
+    for (size_t i = 0; i < rPrimitive2DSequence.size(); i++)
     {
         drawinglayer::primitive2d::Primitive2DReference xPrimitive2DReference = rPrimitive2DSequence[i];
         const BasePrimitive2D* pBasePrimitive = dynamic_cast<const BasePrimitive2D* >(xPrimitive2DReference.get());