From b6423e8be3370c609259b26fb8c05c033bf25ba4 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Thu, 1 Nov 2007 20:41:30 +0000 Subject: [PATCH] don't crash in HiglightAll() (called, in particular, from SortItems()) when the control is empty (patch 1821550) git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@49590 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- src/generic/listctrl.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/generic/listctrl.cpp b/src/generic/listctrl.cpp index 7e1830b570..fbb923ae18 100644 --- a/src/generic/listctrl.cpp +++ b/src/generic/listctrl.cpp @@ -2865,9 +2865,10 @@ void wxListMainWindow::HighlightAll( bool on ) RefreshLine(m_current); } } - else // multi sel + else // multi selection { - HighlightLines(0, GetItemCount() - 1, on); + if ( !IsEmpty() ) + HighlightLines(0, GetItemCount() - 1, on); } } @@ -3024,7 +3025,7 @@ void wxListMainWindow::OnMouse( wxMouseEvent &event ) if (event.RightDown()) { SendNotify( (size_t)-1, wxEVT_COMMAND_LIST_ITEM_RIGHT_CLICK, event.GetPosition() ); - + wxContextMenuEvent evtCtx( wxEVT_CONTEXT_MENU, GetParent()->GetId(), @@ -3107,7 +3108,7 @@ void wxListMainWindow::OnMouse( wxMouseEvent &event ) if (event.RightDown()) { SendNotify( (size_t) -1, wxEVT_COMMAND_LIST_ITEM_RIGHT_CLICK, event.GetPosition() ); - + wxContextMenuEvent evtCtx( wxEVT_CONTEXT_MENU, GetParent()->GetId(), -- 2.45.2