cheng/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Micro optimization in wxListMainWindow::SendNotify()

Browse Source 
Just call GetListCtrl() once instead of calling both it and GetParent()
multiple times.

No real changes.
This commit is contained in:
Vadim Zeitlin 2022-06-08 13:36:47 +01:00
parent 1a545fe428
commit 3eee299cd3
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/generic/listctrl.cpp
View File

@ -2331,8 +2331,10 @@ void wxListMainWindow::SendNotify( size_t line,
wxEventType command,
const wxPoint& point )
{
wxListEvent le( command, GetParent()->GetId() );
le.SetEventObject( GetParent() );
wxGenericListCtrl* const listctrl = GetListCtrl();
wxListEvent le( command, listctrl->GetId() );
le.SetEventObject( listctrl );
le.m_item.m_itemId =
le.m_itemIndex = line;
@ -2349,12 +2351,12 @@ void wxListMainWindow::SendNotify( size_t line,
// provide information about the (first column of the) item in the event if
// we have a valid item and any columns at all
if ( line != (size_t)-1 && GetListCtrl()->GetColumnCount() )
if ( line != (size_t)-1 && listctrl->GetColumnCount() )
{
GetLine(line)->GetItem( 0, le.m_item );
}
GetParent()->GetEventHandler()->ProcessEvent( le );
listctrl->GetEventHandler()->ProcessEvent( le );
}
bool wxListMainWindow::ChangeCurrentWithoutEvent(size_t current)