Replace lists by vectors in printerjob (vcl)

Change-Id: I6eb9b94839307e6297b5c3e3308d95d3645a215f
Reviewed-on: https://gerrit.libreoffice.org/43665
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

2 files changed, 20 insertions, 23 deletions

diff --git a/vcl/inc/unx/printerjob.hxx b/vcl/inc/unx/printerjob.hxx
index db2f437f02a6..bd8015593de9 100644
--- a/vcl/inc/unx/printerjob.hxx
+++ b/vcl/inc/unx/printerjob.hxx
@@ -24,7 +24,7 @@
 #include "osl/file.hxx"
 #include "rtl/string.hxx"
 
-#include <list>
+#include <vector>
 
 namespace psp {
 
@@ -41,8 +41,8 @@ private:
     osl::File*              mpJobHeader;
     osl::File*              mpJobTrailer;
 
-    std::list< osl::File* > maPageList;
-    std::list< osl::File* > maHeaderList;
+    std::vector< osl::File* > maPageVector;
+    std::vector< osl::File* > maHeaderVector;
 
     JobData                 m_aDocumentJobData;
     JobData                 m_aLastJobData;
diff --git a/vcl/unx/generic/print/printerjob.cxx b/vcl/unx/generic/print/printerjob.cxx
index 39ac5c79c425..c3005fca8de7 100644
--- a/vcl/unx/generic/print/printerjob.cxx
+++ b/vcl/unx/generic/print/printerjob.cxx
@@ -158,7 +158,7 @@ PrinterJob::IsColorPrinter () const
 osl::File*
 PrinterJob::GetCurrentPageBody ()
 {
-    return maPageList.back();
+    return maPageVector.back();
 }
 
 /*
@@ -239,16 +239,13 @@ createSpoolDir ()
 
 PrinterJob::~PrinterJob ()
 {
-    std::list< osl::File* >::iterator pPage;
-    for (pPage = maPageList.begin(); pPage != maPageList.end(); ++pPage)
+    for (auto const& page : maPageVector)
     {
-        //(*pPage)->remove();
-        delete *pPage;
+        delete page;
     }
-    for (pPage = maHeaderList.begin(); pPage != maHeaderList.end(); ++pPage)
+    for (auto const& header : maHeaderVector)
     {
-        //(*pPage)->remove();
-        delete *pPage;
+        delete header;
     }
     // mpJobHeader->remove();
     delete mpJobHeader;
@@ -406,7 +403,7 @@ bool
 PrinterJob::EndJob()
 {
     // no pages ? that really means no print job
-    if( maPageList.empty() )
+    if( maPageVector.empty() )
         return false;
 
     // write document setup (done here because it
@@ -429,7 +426,7 @@ PrinterJob::EndJob()
     else
         aTrailer.append("\n%%Orientation: Portrait");
     aTrailer.append( "\n%%Pages: " );
-    aTrailer.append( (sal_Int32)maPageList.size() );
+    aTrailer.append( (sal_Int32)maPageVector.size() );
     aTrailer.append( "\n%%EOF\n" );
     WritePS (mpJobTrailer, aTrailer.getStr());
 
@@ -485,10 +482,10 @@ PrinterJob::EndJob()
     mpJobHeader->close();
 
     bool bSuccess = true;
-    std::list< osl::File* >::iterator pPageBody;
-    std::list< osl::File* >::iterator pPageHead;
-    for (pPageBody  = maPageList.begin(), pPageHead  = maHeaderList.begin();
-         pPageBody != maPageList.end() && pPageHead != maHeaderList.end();
+    std::vector< osl::File* >::iterator pPageBody;
+    std::vector< osl::File* >::iterator pPageHead;
+    for (pPageBody  = maPageVector.begin(), pPageHead  = maHeaderVector.begin();
+         pPageBody != maPageVector.end() && pPageHead != maHeaderVector.end();
          ++pPageBody, ++pPageHead)
     {
         if( *pPageHead )
@@ -573,14 +570,14 @@ PrinterJob::StartPage (const JobData& rJobSetup)
 {
     InitPaperSize (rJobSetup);
 
-    OUString aPageNo = OUString::number ((sal_Int32)maPageList.size()+1); // sequential page number must start with 1
+    OUString aPageNo = OUString::number ((sal_Int32)maPageVector.size()+1); // sequential page number must start with 1
     OUString aExt    = aPageNo + ".ps";
 
     osl::File* pPageHeader = CreateSpoolFile ( "psp_pghead", aExt);
     osl::File* pPageBody   = CreateSpoolFile ( "psp_pgbody", aExt);
 
-    maHeaderList.push_back (pPageHeader);
-    maPageList.push_back (pPageBody);
+    maHeaderVector.emplace_back (pPageHeader);
+    maPageVector.emplace_back (pPageBody);
 
     if( ! (pPageHeader && pPageBody) )
         return;
@@ -624,7 +621,7 @@ PrinterJob::StartPage (const JobData& rJobSetup)
      *  different.
      */
     bool bWriteFeatures = true;
-    if( 1 == maPageList.size() )
+    if( 1 == maPageVector.size() )
     {
         m_aDocumentJobData = rJobSetup;
         bWriteFeatures = false;
@@ -639,8 +636,8 @@ PrinterJob::StartPage (const JobData& rJobSetup)
 bool
 PrinterJob::EndPage ()
 {
-    osl::File* pPageHeader = maHeaderList.back();
-    osl::File* pPageBody   = maPageList.back();
+    osl::File* pPageHeader = maHeaderVector.back();
+    osl::File* pPageBody   = maPageVector.back();
 
     if( ! (pPageBody && pPageHeader) )
         return false;