diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-12 00:27:02 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-12 06:13:26 +0100 |
| commit | 270e76f6eba44749743761c5575adf9f08e84675 (patch) | |
| tree | df46dd982d5954ed778440690fccc755db68a34c /sc/source/ui | |
| parent | 69f279d7fd266c80584eb845291a6ba3df26631b (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: Ifb3c9016019977aa2c51dbd0a0394a0f81150df5
Reviewed-on: https://gerrit.libreoffice.org/63270
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'sc/source/ui')
| -rw-r--r-- | sc/source/ui/unoobj/cellsuno.cxx | 2 | ||||
| -rw-r--r-- | sc/source/ui/view/cellsh1.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sc/source/ui/unoobj/cellsuno.cxx b/sc/source/ui/unoobj/cellsuno.cxx index 15d5a9b2853c..5d318c0d3696 100644 --- a/sc/source/ui/unoobj/cellsuno.cxx +++ b/sc/source/ui/unoobj/cellsuno.cxx @@ -8269,7 +8269,7 @@ void ScTableSheetObj::SetOnePropertyValue( const SfxItemPropertySimpleEntry* pEn else if ( pEntry->nWID == SC_WID_UNO_CODENAME ) { OUString aCodeName; - if ( pDocSh && ( aValue >>= aCodeName ) ) + if (aValue >>= aCodeName) { pDocSh->GetDocument().SetCodeName( GetTab_Impl(), aCodeName ); } diff --git a/sc/source/ui/view/cellsh1.cxx b/sc/source/ui/view/cellsh1.cxx index c57d3e75ac98..0005ae903d63 100644 --- a/sc/source/ui/view/cellsh1.cxx +++ b/sc/source/ui/view/cellsh1.cxx @@ -1370,7 +1370,7 @@ void ScCellShell::ExecuteEdit( SfxRequest& rReq ) // if ChangeTrack MoveMode disable pDlg->SetChangeTrack( pDoc->GetChangeTrack() != nullptr ); // fdo#56098 disable shift if necessary - if ( !bOtherDoc && pOwnClip ) + if (!bOtherDoc) { ScViewData* pData = GetViewData(); if ( pData->GetMarkData().GetTableSelect( pData->GetTabNo() ) ) |