strepsirrhini/office-gobmx
9
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

wina11y: Drop duplicate variable + check

Browse source 
No need to have `pAccessibleContext` as a raw
pointer, just use the `xContext`. And that one
has already been checked for being a non-empty
Reference above.

Change-Id: I15d7e70ddbc8b260e5e1bc0eb4e190c54d6a2ea2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177884
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
This commit is contained in:
Michael Weghorn 2024-12-05 11:42:25 +00:00
parent 6708246e20
commit 070b198961
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
winaccessibility/source/service/AccTopWindowListener.cxx
View file

@ -61,9 +61,8 @@ void AccTopWindowListener::HandleWindowOpened( css::accessibility::XAccessible*
if(!xContext.is())
return;
css::accessibility::XAccessibleContext* pAccessibleContext = xContext.get();
//Only AccessibleContext exist, add all listeners
if(pAccessibleContext != nullptr && systemdata != nullptr)
// add all listeners
if (systemdata != nullptr)
{
m_aAccObjectManager.SaveTopWindowHandle(systemdata->hWnd, pAccessible);
@ -72,7 +71,7 @@ void AccTopWindowListener::HandleWindowOpened( css::accessibility::XAccessible*
if( window->GetStyle() & WB_MOVEABLE )
m_aAccObjectManager.IncreaseState( pAccessible, static_cast<unsigned short>(-1) /* U_MOVEBLE */ );
short role = pAccessibleContext->getAccessibleRole();
short role = xContext->getAccessibleRole();
if (role == css::accessibility::AccessibleRole::POPUP_MENU ||
role == css::accessibility::AccessibleRole::MENU )