From e523b205364e59f11ae3e735c1440d61427f15d7 Mon Sep 17 00:00:00 2001 From: Michael Weghorn Date: Wed, 5 Jan 2022 14:29:18 +0000 Subject: wina11y: Deduplicate 2 common lines in CMAccessible::get_accRole Change-Id: I3bd4f26861d46f6a43a4b0edda3dbf628e2da2cd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128010 Tested-by: Jenkins Reviewed-by: Michael Weghorn --- winaccessibility/source/UAccCOM/MAccessible.cxx | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'winaccessibility') diff --git a/winaccessibility/source/UAccCOM/MAccessible.cxx b/winaccessibility/source/UAccCOM/MAccessible.cxx index 0e3de1af1b53..c905cc82b875 100644 --- a/winaccessibility/source/UAccCOM/MAccessible.cxx +++ b/winaccessibility/source/UAccCOM/MAccessible.cxx @@ -505,18 +505,14 @@ COM_DECLSPEC_NOTHROW STDMETHODIMP CMAccessible::get_accRole(VARIANT varChild, VA if(varChild.lVal == CHILDID_SELF) { - if( m_iRole < IA2_ROLE_CAPTION ) - { - VariantInit(pvarRole); - pvarRole->vt = VT_I4; + VariantInit(pvarRole); + pvarRole->vt = VT_I4; + + if (m_iRole < IA2_ROLE_CAPTION) pvarRole->lVal = m_iRole; - } else - { - VariantInit(pvarRole); - pvarRole->vt = VT_I4; pvarRole->lVal = ROLE_SYSTEM_CLIENT; - } + return S_OK; } -- cgit