diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-11-05 15:20:22 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-11-06 06:27:51 +0100 |
| commit | 7b99cdb2d77a3feaf9b782c1e656f2d922e2746e (patch) | |
| tree | 180459dcd8379ee1a2e89098a4d9eda72abe8f05 /unotools | |
| parent | 7ee07296a66df29555c9e9a684f24bc68201cb78 (diff) | |
loplugin:indentation find broken if statements
so I don't read the "then" block as being a sequential statements
Change-Id: Ib2004acd3518bd4ebd2246f02a26c2c0a8bbab4c
Reviewed-on: https://gerrit.libreoffice.org/82069
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'unotools')
| -rw-r--r-- | unotools/source/config/configitem.cxx | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/unotools/source/config/configitem.cxx b/unotools/source/config/configitem.cxx index fd12fbde6796..8ed1d317a09c 100644 --- a/unotools/source/config/configitem.cxx +++ b/unotools/source/config/configitem.cxx @@ -873,26 +873,26 @@ bool ConfigItem::ReplaceSetProperties( { bool bFound = comphelper::findValue(aSubNodeNames, rContainerSubNode) != -1; if(!bFound) - try - { - xCont->removeByName(rContainerSubNode); - } - catch (const Exception&) - { - if (isSimpleValueSet) + try { - try + xCont->removeByName(rContainerSubNode); + } + catch (const Exception&) + { + if (isSimpleValueSet) { - // #i37322#: fallback action: replace with <void/> - xCont->replaceByName(rContainerSubNode, Any()); - // fallback successful: continue looping - continue; + try + { + // #i37322#: fallback action: replace with <void/> + xCont->replaceByName(rContainerSubNode, Any()); + // fallback successful: continue looping + continue; + } + catch (Exception &) + {} // propagate original exception, if fallback fails } - catch (Exception &) - {} // propagate original exception, if fallback fails + throw; } - throw; - } } try { xBatch->commitChanges(); } catch (css::uno::Exception &) |