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 /chart2 | |
| 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 'chart2')
| -rw-r--r-- | chart2/source/view/main/PropertyMapper.cxx | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/chart2/source/view/main/PropertyMapper.cxx b/chart2/source/view/main/PropertyMapper.cxx index c4b4b6756097..f96c7b101509 100644 --- a/chart2/source/view/main/PropertyMapper.cxx +++ b/chart2/source/view/main/PropertyMapper.cxx @@ -405,7 +405,9 @@ void PropertyMapper::setMultiProperties( TOOLS_WARN_EXCEPTION("chart2", "" ); //if this occurs more often think of removing the XMultiPropertySet completely for better performance } - if(!bSuccess) + if(bSuccess) + return; + try { sal_Int32 nCount = std::max( rNames.getLength(), rValues.getLength() ); |