diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-30 15:34:20 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-31 12:42:50 +0200 |
| commit | 714fe0fed88d01bac1a658fbb2de193a7704e24b (patch) | |
| tree | 48d669b9555ff78dba0c87e9206db030e39cb499 /cui | |
| parent | cb9ce13d82839a7ce0db3bd113e70bbcc125076a (diff) | |
loplugin:returnconstant in svx
Change-Id: I0cc0c32afa0d735738c6744873f3518a2763821e
Reviewed-on: https://gerrit.libreoffice.org/58334
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'cui')
| -rw-r--r-- | cui/source/tabpages/numfmt.cxx | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/cui/source/tabpages/numfmt.cxx b/cui/source/tabpages/numfmt.cxx index 2aa9505507e6..339451961612 100644 --- a/cui/source/tabpages/numfmt.cxx +++ b/cui/source/tabpages/numfmt.cxx @@ -1387,11 +1387,12 @@ bool SvxNumberFormatTabPage::Click_Impl(PushButton* pIB) if(bOneAreaFlag && (nFixedCategory!=nCatLbSelPos)) { if(bAdded) aEntryList.clear(); - bDeleted = pNumFmtShell->RemoveFormat( aFormat, - nCatLbSelPos, - nFmtLbSelPos, - a2EntryList); - if(bDeleted) a2EntryList.clear(); + pNumFmtShell->RemoveFormat( aFormat, + nCatLbSelPos, + nFmtLbSelPos, + a2EntryList); + bDeleted = true; + a2EntryList.clear(); m_pEdFormat->GrabFocus(); m_pEdFormat->SetSelection( Selection( 0, SELECTION_MAX ) ); nReturn |= nReturnOneArea; @@ -1444,10 +1445,11 @@ bool SvxNumberFormatTabPage::Click_Impl(PushButton* pIB) sal_uInt16 nCatLbSelPos = 0; short nFmtLbSelPos = SELPOS_NONE; - bDeleted = pNumFmtShell->RemoveFormat( aFormat, - nCatLbSelPos, - nFmtLbSelPos, - aEntryList ); + pNumFmtShell->RemoveFormat( aFormat, + nCatLbSelPos, + nFmtLbSelPos, + aEntryList ); + bDeleted = true; m_pEdComment->SetText(m_pLbCategory->GetEntry(1)); if ( bDeleted ) |