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 /vcl | |
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 'vcl')
-rw-r--r-- | vcl/source/window/dialog.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vcl/source/window/dialog.cxx b/vcl/source/window/dialog.cxx index b426ad39a613..d91f5aea30f0 100644 --- a/vcl/source/window/dialog.cxx +++ b/vcl/source/window/dialog.cxx @@ -148,7 +148,7 @@ vcl::Window * nextLogicalChildOfParent(const vcl::Window *pTopLevel, const vcl:: pChild = pParent->GetWindow(GetWindowType::Next); } - if (pChild && isContainerWindow(*pChild)) + if (isContainerWindow(*pChild)) pChild = nextLogicalChildOfParent(pTopLevel, pChild); return const_cast<vcl::Window *>(pChild); @@ -176,7 +176,7 @@ vcl::Window * prevLogicalChildOfParent(const vcl::Window *pTopLevel, const vcl:: pChild = pParent->GetWindow(GetWindowType::Prev); } - if (pChild && isContainerWindow(*pChild)) + if (isContainerWindow(*pChild)) pChild = prevLogicalChildOfParent(pTopLevel, pChild); return const_cast<vcl::Window *>(pChild); |