summaryrefslogtreecommitdiff
path: root/sfx2
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2018-02-28 14:37:10 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2018-03-05 07:27:31 +0100
commitc9f01cc7855a20b1e0bff662cfbb75fe3665cd8a (patch)
tree16a303215b6de2fa1ac9352f8be4a67203b70048 /sfx2
parent260a443be052ba2c0c57584130a638341a2d3014 (diff)
loplugin:useuniqueptr in StyleTree_Impl
Change-Id: I0541283ad9b5719f0b181ba1bdedeb287316c8a2 Reviewed-on: https://gerrit.libreoffice.org/50661 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sfx2')
-rw-r--r--sfx2/source/control/objface.cxx19
-rw-r--r--sfx2/source/dialog/templdlg.cxx32
2 files changed, 18 insertions, 33 deletions
diff --git a/sfx2/source/control/objface.cxx b/sfx2/source/control/objface.cxx
index 244c8781fee5..78f561e3cedf 100644
--- a/sfx2/source/control/objface.cxx
+++ b/sfx2/source/control/objface.cxx
@@ -73,8 +73,10 @@ typedef std::vector<SfxObjectUI_Impl*> SfxObjectUIArr_Impl;
struct SfxInterface_Impl
{
- SfxObjectUIArr_Impl aObjectBars; // registered ObjectBars
- SfxObjectUIArr_Impl aChildWindows; // registered ChildWindows
+ std::vector<std::unique_ptr<SfxObjectUI_Impl>>
+ aObjectBars; // registered ObjectBars
+ std::vector<std::unique_ptr<SfxObjectUI_Impl>>
+ aChildWindows; // registered ChildWindows
OUString aPopupName; // registered PopupMenu
StatusBarId eStatBarResId; // registered StatusBar
SfxModule* pModule;
@@ -86,15 +88,6 @@ struct SfxInterface_Impl
, bRegistered(false)
{
}
-
- ~SfxInterface_Impl()
- {
- for (auto const& objectBar : aObjectBars)
- delete objectBar;
-
- for (auto const& childWindow : aChildWindows)
- delete childWindow;
- }
};
static SfxObjectUI_Impl* CreateObjectBarUI_Impl(sal_uInt16 nPos, SfxVisibilityFlags nFlags, ToolbarId eId, SfxShellFeature nFeature);
@@ -280,7 +273,7 @@ void SfxInterface::RegisterObjectBar(sal_uInt16 nPos, SfxVisibilityFlags nFlags,
{
SfxObjectUI_Impl* pUI = CreateObjectBarUI_Impl(nPos, nFlags, eId, nFeature);
if ( pUI )
- pImplData->aObjectBars.push_back(pUI);
+ pImplData->aObjectBars.emplace_back(pUI);
}
SfxObjectUI_Impl* CreateObjectBarUI_Impl(sal_uInt16 nPos, SfxVisibilityFlags nFlags, ToolbarId eId, SfxShellFeature nFeature)
@@ -365,7 +358,7 @@ void SfxInterface::RegisterChildWindow(sal_uInt16 nId, bool bContext, SfxShellFe
{
SfxObjectUI_Impl* pUI = new SfxObjectUI_Impl(0, SfxVisibilityFlags::Invisible, nId, nFeature);
pUI->bContext = bContext;
- pImplData->aChildWindows.push_back(pUI);
+ pImplData->aChildWindows.emplace_back(pUI);
}
void SfxInterface::RegisterStatusBar(StatusBarId eId)
diff --git a/sfx2/source/dialog/templdlg.cxx b/sfx2/source/dialog/templdlg.cxx
index 412b02f35110..7f1a9a77038b 100644
--- a/sfx2/source/dialog/templdlg.cxx
+++ b/sfx2/source/dialog/templdlg.cxx
@@ -499,7 +499,7 @@ void StyleTreeListBox_Impl::Recalc()
/** Internal structure for the establishment of the hierarchical view */
class StyleTree_Impl;
-typedef std::vector<StyleTree_Impl*> StyleTreeArr_Impl;
+typedef std::vector<std::unique_ptr<StyleTree_Impl>> StyleTreeArr_Impl;
class StyleTree_Impl
{
@@ -513,20 +513,14 @@ public:
StyleTree_Impl(const OUString &rName, const OUString &rParent):
aName(rName), aParent(rParent), pChildren(0) {}
- ~StyleTree_Impl();
const OUString& getName() { return aName; }
const OUString& getParent() { return aParent; }
StyleTreeArr_Impl& getChildren() { return pChildren; }
};
-StyleTree_Impl::~StyleTree_Impl()
-{
- for (auto const& child : pChildren)
- delete child;
-}
-StyleTreeArr_Impl& MakeTree_Impl(StyleTreeArr_Impl& rArr)
+void MakeTree_Impl(StyleTreeArr_Impl& rArr)
{
const comphelper::string::NaturalStringSorter aSorter(
::comphelper::getProcessComponentContext(),
@@ -536,11 +530,11 @@ StyleTreeArr_Impl& MakeTree_Impl(StyleTreeArr_Impl& rArr)
styleFinder.reserve(rArr.size());
for (const auto& pEntry : rArr)
{
- styleFinder.emplace(pEntry->getName(), pEntry);
+ styleFinder.emplace(pEntry->getName(), pEntry.get());
}
// Arrange all under their Parents
- for (const auto& pEntry : rArr)
+ for (auto& pEntry : rArr)
{
if (!pEntry->HasParent())
continue;
@@ -550,21 +544,19 @@ StyleTreeArr_Impl& MakeTree_Impl(StyleTreeArr_Impl& rArr)
StyleTree_Impl* pCmp = it->second;
// Insert child entries sorted
auto iPos = std::lower_bound(pCmp->getChildren().begin(), pCmp->getChildren().end(), pEntry,
- [&aSorter](StyleTree_Impl* pEntry1, StyleTree_Impl* pEntry2) { return aSorter.compare(pEntry1->getName(), pEntry2->getName()) < 0; });
- pCmp->getChildren().insert(iPos, pEntry);
+ [&aSorter](std::unique_ptr<StyleTree_Impl> const & pEntry1, std::unique_ptr<StyleTree_Impl> const & pEntry2) { return aSorter.compare(pEntry1->getName(), pEntry2->getName()) < 0; });
+ pCmp->getChildren().insert(iPos, std::move(pEntry));
}
}
// Only keep tree roots in rArr, child elements can be accessed through the hierarchy
- rArr.erase(std::remove_if(rArr.begin(), rArr.end(), [](StyleTree_Impl* pEntry) { return pEntry->HasParent(); }), rArr.end());
+ rArr.erase(std::remove_if(rArr.begin(), rArr.end(), [](std::unique_ptr<StyleTree_Impl> const & pEntry) { return !pEntry; }), rArr.end());
// tdf#91106 sort top level styles
std::sort(rArr.begin(), rArr.end(),
- [&aSorter](StyleTree_Impl* pEntry1, StyleTree_Impl* pEntry2) {
+ [&aSorter](std::unique_ptr<StyleTree_Impl> const & pEntry1, std::unique_ptr<StyleTree_Impl> const & pEntry2) {
return aSorter.compare(pEntry1->getName(), pEntry2->getName()) < 0;
});
-
- return rArr;
}
inline bool IsExpanded_Impl( const ExpandedEntries_t& rEntries,
@@ -595,7 +587,7 @@ SvTreeListEntry* FillBox_Impl(SvTreeListBox* pBox,
for(size_t i = 0; i < pEntry->getChildren().size(); ++i)
{
- FillBox_Impl(pBox, pEntry->getChildren()[i], rEntries, eStyleFamily, pTreeListEntry);
+ FillBox_Impl(pBox, pEntry->getChildren()[i].get(), rEntries, eStyleFamily, pTreeListEntry);
}
return pTreeListEntry;
}
@@ -1033,7 +1025,7 @@ void SfxCommonTemplateDialog_Impl::FillTreeBox()
while (pStyle)
{
StyleTree_Impl* pNew = new StyleTree_Impl(pStyle->GetName(), pStyle->GetParent());
- aArr.push_back(pNew);
+ aArr.emplace_back(pNew);
pStyle = pStyleSheetPool->Next();
}
@@ -1046,8 +1038,8 @@ void SfxCommonTemplateDialog_Impl::FillTreeBox()
for (sal_uInt16 i = 0; i < nCount; ++i)
{
- FillBox_Impl(pTreeBox, aArr[i], aEntries, pItem->GetFamily(), nullptr);
- delete aArr[i];
+ FillBox_Impl(pTreeBox, aArr[i].get(), aEntries, pItem->GetFamily(), nullptr);
+ aArr[i].reset();
}
pTreeBox->Recalc();