diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-13 00:40:43 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-18 12:32:18 +0100 |
| commit | a208ece66416048cdc99235a569bd8c928a07394 (patch) | |
| tree | e05fd7f828564bd370d21f897cfd2da7fa8f7488 /sd | |
| parent | cf6b3bece7d46878302075329c72c7541c65bbd0 (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I60bb778a88dd5619efd4858eb74d56d28616c777
Reviewed-on: https://gerrit.libreoffice.org/63520
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'sd')
| -rw-r--r-- | sd/source/ui/sidebar/DocumentHelper.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sd/source/ui/sidebar/DocumentHelper.cxx b/sd/source/ui/sidebar/DocumentHelper.cxx index f48065ca6923..9392d5fa4ab6 100644 --- a/sd/source/ui/sidebar/DocumentHelper.cxx +++ b/sd/source/ui/sidebar/DocumentHelper.cxx @@ -95,8 +95,7 @@ SdPage* DocumentHelper::CopyMasterPageToLocalDocument ( for (sal_uInt16 nMaster=0; nMaster<nMasterPageCount; nMaster++) { SdPage* pCandidate = rTargetDocument.GetMasterSdPage (nMaster, PageKind::Standard); - if (pMasterPage!=nullptr - && pCandidate->GetName() == pMasterPage->GetName()) + if (pCandidate->GetName() == pMasterPage->GetName()) { bPageExists = true; pNewMasterPage = pCandidate; |