From a249cfc2bcc7f4e5784b47581327c99e551d0e55 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 10 Nov 2015 14:42:13 +0200 Subject: editeng: boost::ptr_vector->std::vector Change-Id: I562b1f7cd1e5e7f7fc03b8f54bd6117255703eb5 --- editeng/source/editeng/editdoc.cxx | 45 ++++++++++++++++++++----------------- editeng/source/editeng/editdoc.hxx | 5 ++--- editeng/source/editeng/editundo.hxx | 1 + 3 files changed, 27 insertions(+), 24 deletions(-) (limited to 'editeng') diff --git a/editeng/source/editeng/editdoc.cxx b/editeng/source/editeng/editdoc.cxx index 31a5786a2dc1..9b484a3079f7 100644 --- a/editeng/source/editeng/editdoc.cxx +++ b/editeng/source/editeng/editdoc.cxx @@ -708,7 +708,7 @@ sal_Int32 FastGetPos(const _Array& rArray, const _Val* p, sal_Int32& rLastPos) for (sal_Int32 nIdx = rLastPos - 2; nIdx < nEnd; ++nIdx) { - if (&rArray.at(nIdx) == p) + if (rArray.at(nIdx).get() == p) { rLastPos = nIdx; return nIdx; @@ -717,7 +717,7 @@ sal_Int32 FastGetPos(const _Array& rArray, const _Val* p, sal_Int32& rLastPos) } // The world's lamest linear search from svarray... for (sal_Int32 nIdx = 0; nIdx < nArrayLen; ++nIdx) - if (&rArray.at(nIdx) == p) + if (rArray.at(nIdx).get() == p) return rLastPos = nIdx; // XXX "not found" condition for sal_Int32 indexes @@ -741,12 +741,12 @@ sal_Int32 ParaPortionList::GetPos(const ParaPortion* p) const ParaPortion* ParaPortionList::operator [](sal_Int32 nPos) { - return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? &maPortions[nPos] : nullptr; + return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? maPortions[nPos].get() : nullptr; } const ParaPortion* ParaPortionList::operator [](sal_Int32 nPos) const { - return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? &maPortions[nPos] : nullptr; + return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? maPortions[nPos].get() : nullptr; } ParaPortion* ParaPortionList::Release(sal_Int32 nPos) @@ -756,7 +756,9 @@ ParaPortion* ParaPortionList::Release(sal_Int32 nPos) SAL_WARN( "editeng", "ParaPortionList::Release - out of bounds pos " << nPos); return nullptr; } - return maPortions.release(maPortions.begin()+nPos).release(); + ParaPortion* p = maPortions[nPos].release(); + maPortions.erase(maPortions.begin()+nPos); + return p; } void ParaPortionList::Remove(sal_Int32 nPos) @@ -776,12 +778,12 @@ void ParaPortionList::Insert(sal_Int32 nPos, ParaPortion* p) SAL_WARN( "editeng", "ParaPortionList::Insert - out of bounds pos " << nPos); return; } - maPortions.insert(maPortions.begin()+nPos, p); + maPortions.insert(maPortions.begin()+nPos, std::unique_ptr(p)); } void ParaPortionList::Append(ParaPortion* p) { - maPortions.push_back(p); + maPortions.push_back(std::unique_ptr(p)); } sal_Int32 ParaPortionList::Count() const @@ -805,7 +807,7 @@ long ParaPortionList::GetYOffset(const ParaPortion* pPPortion) const long nHeight = 0; for (sal_Int32 i = 0, n = maPortions.size(); i < n; ++i) { - const ParaPortion* pTmpPortion = &maPortions[i]; + const ParaPortion* pTmpPortion = maPortions[i].get(); if ( pTmpPortion == pPPortion ) return nHeight; nHeight += pTmpPortion->GetHeight(); @@ -819,7 +821,7 @@ sal_Int32 ParaPortionList::FindParagraph(long nYOffset) const long nY = 0; for (size_t i = 0, n = maPortions.size(); i < n; ++i) { - nY += maPortions[i].GetHeight(); // should also be correct even in bVisible! + nY += maPortions[i]->GetHeight(); // should also be correct even in bVisible! if ( nY > nYOffset ) return i <= SAL_MAX_INT32 ? static_cast(i) : SAL_MAX_INT32; } @@ -828,12 +830,12 @@ sal_Int32 ParaPortionList::FindParagraph(long nYOffset) const const ParaPortion* ParaPortionList::SafeGetObject(sal_Int32 nPos) const { - return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? &maPortions[nPos] : nullptr; + return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? maPortions[nPos].get() : nullptr; } ParaPortion* ParaPortionList::SafeGetObject(sal_Int32 nPos) { - return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? &maPortions[nPos] : nullptr; + return 0 <= nPos && nPos < (sal_Int32)maPortions.size() ? maPortions[nPos].get() : nullptr; } #if OSL_DEBUG_LEVEL > 0 @@ -1982,22 +1984,22 @@ EditDoc::~EditDoc() namespace { -class RemoveEachItemFromPool : std::unary_function +class RemoveEachItemFromPool : std::unary_function, void> { EditDoc& mrDoc; public: explicit RemoveEachItemFromPool(EditDoc& rDoc) : mrDoc(rDoc) {} - void operator() (const ContentNode& rNode) + void operator() (const std::unique_ptr& rNode) { - mrDoc.RemoveItemsFromPool(rNode); + mrDoc.RemoveItemsFromPool(*rNode.get()); } }; -struct ClearSpellErrorsHandler : std::unary_function +struct ClearSpellErrorsHandler : std::unary_function, void> { - void operator() (ContentNode& rNode) + void operator() (std::unique_ptr& rNode) { - rNode.DestroyWrongList(); + rNode->DestroyWrongList(); } }; @@ -2118,12 +2120,12 @@ sal_Int32 EditDoc::GetPos(const ContentNode* p) const const ContentNode* EditDoc::GetObject(sal_Int32 nPos) const { - return 0 <= nPos && nPos < (sal_Int32)maContents.size() ? &maContents[nPos] : nullptr; + return 0 <= nPos && nPos < (sal_Int32)maContents.size() ? maContents[nPos].get() : nullptr; } ContentNode* EditDoc::GetObject(sal_Int32 nPos) { - return 0 <= nPos && nPos < (sal_Int32)maContents.size() ? &maContents[nPos] : nullptr; + return 0 <= nPos && nPos < (sal_Int32)maContents.size() ? maContents[nPos].get() : nullptr; } const ContentNode* EditDoc::operator[](sal_Int32 nPos) const @@ -2143,7 +2145,7 @@ void EditDoc::Insert(sal_Int32 nPos, ContentNode* p) SAL_WARN( "editeng", "EditDoc::Insert - overflow pos " << nPos); return; } - maContents.insert(maContents.begin()+nPos, p); + maContents.insert(maContents.begin()+nPos, std::unique_ptr(p)); } void EditDoc::Remove(sal_Int32 nPos) @@ -2163,7 +2165,8 @@ void EditDoc::Release(sal_Int32 nPos) SAL_WARN( "editeng", "EditDoc::Release - out of bounds pos " << nPos); return; } - maContents.release(maContents.begin() + nPos).release(); + maContents[nPos].release(); + maContents.erase(maContents.begin() + nPos); } sal_Int32 EditDoc::Count() const diff --git a/editeng/source/editeng/editdoc.hxx b/editeng/source/editeng/editdoc.hxx index 55754f6ebeae..5c419cd5c4a1 100644 --- a/editeng/source/editeng/editdoc.hxx +++ b/editeng/source/editeng/editdoc.hxx @@ -35,7 +35,6 @@ #include #include -#include #include class ImpEditEngine; @@ -655,7 +654,7 @@ public: class ParaPortionList { mutable sal_Int32 nLastCache; - boost::ptr_vector maPortions; + std::vector> maPortions; public: ParaPortionList(); ~ParaPortionList(); @@ -743,7 +742,7 @@ class EditDoc { private: mutable sal_Int32 nLastCache; - boost::ptr_vector maContents; + std::vector > maContents; SfxItemPool* pItemPool; Link aModifyHdl; diff --git a/editeng/source/editeng/editundo.hxx b/editeng/source/editeng/editundo.hxx index 775f5a926549..411633c216e6 100644 --- a/editeng/source/editeng/editundo.hxx +++ b/editeng/source/editeng/editundo.hxx @@ -23,6 +23,7 @@ #include #include #include +#include class EditEngine; class EditView; -- cgit v1.2.3