loop with erase(begin()) looks sort of inefficient

xmloff/source/style/xmlnumfi.cxx

@@ -72,7 +72,7 @@ struct SvXMLNumFmtEntry
         aName(rN), nKey(nK), bRemoveAfterUse(bR) {}
 };
 
-class SvXMLNumFmtEntryArr : public boost::ptr_vector<SvXMLNumFmtEntry> {};
+typedef ::boost::ptr_vector<SvXMLNumFmtEntry> SvXMLNumFmtEntryArr;
 
 struct SvXMLEmbeddedElement
 {

xmloff/source/text/XMLTextColumnsContext.cxx

@@ -342,11 +342,10 @@ XMLTextColumnsContext::~XMLTextColumnsContext()
 {
     if( pColumns )
     {
-        while( !pColumns->empty() )
+        for (XMLTextColumnsArray_Impl::iterator it = pColumns->begin();
+                it != pColumns->end(); ++it)
         {
-            XMLTextColumnContext_Impl *pColumn = *pColumns->begin();
-            pColumns->erase( pColumns->begin() );
-            pColumn->ReleaseRef();
+           (*it)->ReleaseRef();
         }
     }
     if( pColumnSep )