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 /sfx2/source | |
| parent | 5690acdb44e8017055bb88e86e19d207836bb95c (diff) | |
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 'sfx2/source')
| -rw-r--r-- | sfx2/source/appl/appserv.cxx | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/sfx2/source/appl/appserv.cxx b/sfx2/source/appl/appserv.cxx index 6bf964c8a98a..2a95052903f2 100644 --- a/sfx2/source/appl/appserv.cxx +++ b/sfx2/source/appl/appserv.cxx @@ -1198,9 +1198,10 @@ void SfxApplication::MiscState_Impl(SfxItemSet &rSet) if ( xLayoutManager.is() ) { - bool bState = true; - bState = xLayoutManager->getElement( "private:resource/menubar/menubar" ).is() - && xLayoutManager->isElementVisible( "private:resource/menubar/menubar" ); + const bool bState + = xLayoutManager->getElement("private:resource/menubar/menubar").is() + && xLayoutManager->isElementVisible( + "private:resource/menubar/menubar"); SfxBoolItem aItem( SID_MENUBAR, bState ); rSet.Put( aItem ); |