diff options
| author | Norbert Thiebaud <nthiebaud@gmail.com> | 2014-01-19 15:07:28 -0600 |
|---|---|---|
| committer | Norbert Thiebaud <nthiebaud@gmail.com> | 2014-01-19 15:07:28 -0600 |
| commit | 9fd10cca8e8a3121b378b9e820c565c101985809 (patch) | |
| tree | 81646c7ecf5ee973590ef2438b956450ff5540e1 /accessibility/source | |
| parent | 014a0e5d6f1acf54c97ac4c21d7a1d72f3f196cf (diff) | |
coverity#1132661 Dereference after null check
Change-Id: Ida147599810b0aff4a7ebd00b90475b1c6cd1366
Diffstat (limited to 'accessibility/source')
| -rw-r--r-- | accessibility/source/standard/vclxaccessiblelist.cxx | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/accessibility/source/standard/vclxaccessiblelist.cxx b/accessibility/source/standard/vclxaccessiblelist.cxx index bc9b71c90bec..f9989047c610 100644 --- a/accessibility/source/standard/vclxaccessiblelist.cxx +++ b/accessibility/source/standard/vclxaccessiblelist.cxx @@ -841,21 +841,18 @@ void VCLXAccessibleList::UpdateSelection_Impl(sal_uInt16) aOldValue <<= getAccessibleChild( (sal_Int32)m_nLastSelectedPos ); aNewValue <<= xNewAcc; } + if (m_pListBoxHelper->IsInDropDown()) + { + if ( aNewValue.hasValue() || aOldValue.hasValue() ) + NotifyAccessibleEvent( + AccessibleEventId::ACTIVE_DESCENDANT_CHANGED, + aOldValue, + aNewValue ); + //the SELECTION_CHANGED is not necessary + //NotifyAccessibleEvent( AccessibleEventId::SELECTION_CHANGED, Any(), Any() ); + } } } - if (!m_pListBoxHelper->IsInDropDown()) - { - } - else - { - if ( aNewValue.hasValue() || aOldValue.hasValue() ) - NotifyAccessibleEvent( - AccessibleEventId::ACTIVE_DESCENDANT_CHANGED, - aOldValue, - aNewValue ); - //the SELECTION_CHANGED is not necessary - //NotifyAccessibleEvent( AccessibleEventId::SELECTION_CHANGED, Any(), Any() ); - } } // ----------------------------------------------------------------------------- |