diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-27 03:16:18 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-27 09:23:52 +0200 |
| commit | dc1858783e72bba808dbd87b5b03d5170449ad52 (patch) | |
| tree | 5164686a8f38e64420f79b027900507d2c179a6d /cppuhelper | |
| parent | tdf#120453 Preserve literal '&' in Qt menu texts (diff) | |
| download | core-dc1858783e72bba808dbd87b5b03d5170449ad52.tar.gz core-dc1858783e72bba808dbd87b5b03d5170449ad52.zip | |
tdf#120703 (PVS): V519 The variable is assigned values twice successively
Change-Id: I9265425a215609ef6bf4298ba39c8399f215ce27
Reviewed-on: https://gerrit.libreoffice.org/62406
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'cppuhelper')
| -rw-r--r-- | cppuhelper/source/propshlp.cxx | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/cppuhelper/source/propshlp.cxx b/cppuhelper/source/propshlp.cxx index 46494e00c827..1a54edffe40e 100644 --- a/cppuhelper/source/propshlp.cxx +++ b/cppuhelper/source/propshlp.cxx @@ -756,11 +756,7 @@ void OPropertySetHelper::fire if( !bVetoable ) { - OInterfaceContainerHelper * pCont = nullptr; - pCont = rBHelper.aLC.getContainer( - getPropertiesTypeIdentifier( ) - ); - if( pCont ) + if (auto pCont = rBHelper.aLC.getContainer(getPropertiesTypeIdentifier())) { // Here is a Bug, unbound properties are also fired OInterfaceIteratorHelper aIt( *pCont ); |