mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cid#1453996 Unchecked return value

Browse Source 
Change-Id: I0b5010aec8ad941d662e851833923a8b6b0a383d
Reviewed-on: https://gerrit.libreoffice.org/79672
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
This commit is contained in:
Caolán McNamara
2019-09-27 09:02:38 +01:00
parent 86cca6b40c
commit 572bd79771
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
cui/source/options/treeopt.cxx
View File

@@ -575,15 +575,14 @@ OfaTreeOptionsDialog::~OfaTreeOptionsDialog()
OptionsPageInfo* OfaTreeOptionsDialog::AddTabPage( OptionsPageInfo* OfaTreeOptionsDialog::AddTabPage(
sal_uInt16 nId, const OUString& rPageName, sal_uInt16 nGroup ) sal_uInt16 nId, const OUString& rPageName, sal_uInt16 nGroup )
{ {
OptionsPageInfo* pPageInfo = new OptionsPageInfo( nId );
std::unique_ptr<weld::TreeIter> xParent = xTreeLB->make_iterator(); std::unique_ptr<weld::TreeIter> xParent = xTreeLB->make_iterator();
xTreeLB->get_iter_first(*xParent); if (!xTreeLB->get_iter_first(*xParent))
return nullptr;
xTreeLB->iter_nth_sibling(*xParent, nGroup); xTreeLB->iter_nth_sibling(*xParent, nGroup);
OptionsPageInfo* pPageInfo = new OptionsPageInfo( nId );
OUString sId(OUString::number(reinterpret_cast<sal_Int64>(pPageInfo)));; OUString sId(OUString::number(reinterpret_cast<sal_Int64>(pPageInfo)));;
xTreeLB->insert(xParent.get(), -1, &rPageName, &sId, nullptr, nullptr, nullptr, false, nullptr); xTreeLB->insert(xParent.get(), -1, &rPageName, &sId, nullptr, nullptr, nullptr, false, nullptr);
return pPageInfo; return pPageInfo;
} }