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 /svx | |
| 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 'svx')
| -rw-r--r-- | svx/source/accessibility/ChildrenManagerImpl.cxx | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/svx/source/accessibility/ChildrenManagerImpl.cxx b/svx/source/accessibility/ChildrenManagerImpl.cxx index 74a956befb7e..ce77aed85e13 100644 --- a/svx/source/accessibility/ChildrenManagerImpl.cxx +++ b/svx/source/accessibility/ChildrenManagerImpl.cxx @@ -783,12 +783,12 @@ AccessibleControlShape * ChildrenManagerImpl::GetAccControlShapeFromModel(css::b for (sal_Int32 index=0;index<count;index++) { AccessibleShape* pAccShape = maVisibleChildren[index].GetAccessibleShape(); - if (pAccShape && ::accessibility::ShapeTypeHandler::Instance().GetTypeId (pAccShape->GetXShape()) == DRAWING_CONTROL) - { - ::accessibility::AccessibleControlShape *pCtlAccShape = static_cast < ::accessibility::AccessibleControlShape* >(pAccShape); - if (pCtlAccShape && pCtlAccShape->GetControlModel() == pSet) + if (pAccShape && ::accessibility::ShapeTypeHandler::Instance().GetTypeId(pAccShape->GetXShape()) == DRAWING_CONTROL) + { + auto* pCtlAccShape = static_cast<::accessibility::AccessibleControlShape*>(pAccShape); + if (pCtlAccShape->GetControlModel() == pSet) return pCtlAccShape; - } + } } return nullptr; } |