diff options
author | Bjoern Michaelsen <bjoern.michaelsen@canonical.com> | 2015-03-28 17:07:28 +0100 |
---|---|---|
committer | Bjoern Michaelsen <bjoern.michaelsen@canonical.com> | 2015-03-28 22:14:37 +0100 |
commit | 9196885f79d261d375036159a86aa47502093587 (patch) | |
tree | b959a002d641fa1071d1bbaacdfe31e0e515e336 /sw | |
parent | 94db1357b1e50ece5b9ac1c5a123ce656aba1790 (diff) |
flatten and simplify
Change-Id: I4da01d589ba170e230aaa4db5d1336c5f93ad168
Diffstat (limited to 'sw')
-rw-r--r-- | sw/source/core/unocore/unotbl.cxx | 42 |
1 files changed, 19 insertions, 23 deletions
diff --git a/sw/source/core/unocore/unotbl.cxx b/sw/source/core/unocore/unotbl.cxx index a05f063f3405..9f532f3c4577 100644 --- a/sw/source/core/unocore/unotbl.cxx +++ b/sw/source/core/unocore/unotbl.cxx @@ -1629,32 +1629,28 @@ sal_Bool SwXTextTableCursor::mergeRange() throw (uno::RuntimeException, std::exception) { SolarMutexGuard aGuard; - bool bRet = false; SwUnoCrsr* pUnoCrsr = GetCrsr(); - if(pUnoCrsr) + if(!pUnoCrsr) + return false; { - { - // The Actions need to be revoked here - UnoActionRemoveContext aRemoveContext(pUnoCrsr->GetDoc()); - } - SwUnoTableCrsr& rTblCrsr = dynamic_cast<SwUnoTableCrsr&>(*pUnoCrsr); - rTblCrsr.MakeBoxSels(); - - { - UnoActionContext aContext(pUnoCrsr->GetDoc()); - bRet = TBLMERGE_OK == rTblCrsr.GetDoc()->MergeTbl(rTblCrsr); - if(bRet) - { - size_t nCount = rTblCrsr.GetSelectedBoxesCount(); - while (nCount--) - { - rTblCrsr.DeleteBox(nCount); - } - } - } - rTblCrsr.MakeBoxSels(); + // The Actions need to be revoked here + UnoActionRemoveContext aRemoveContext(pUnoCrsr->GetDoc()); } - return bRet; + SwUnoTableCrsr& rTblCrsr = dynamic_cast<SwUnoTableCrsr&>(*pUnoCrsr); + rTblCrsr.MakeBoxSels(); + bool bResult; + { + UnoActionContext aContext(pUnoCrsr->GetDoc()); + bResult = TBLMERGE_OK == rTblCrsr.GetDoc()->MergeTbl(rTblCrsr); + } + if(bResult) + { + size_t nCount = rTblCrsr.GetSelectedBoxesCount(); + while (nCount--) + rTblCrsr.DeleteBox(nCount); + } + rTblCrsr.MakeBoxSels(); + return bResult; } sal_Bool SwXTextTableCursor::splitRange(sal_Int16 Count, sal_Bool Horizontal) |