From f45e3ca339c4364d2646da00780377e7a8b88601 Mon Sep 17 00:00:00 2001 From: Marek Roszko Date: Wed, 24 Jan 2024 20:15:23 -0500 Subject: [PATCH] Fix assert in SYMBOL_VIEWER_FRAME::onSelectNextSymbol due to bad current selection check Fixes sentry KICAD-6KE (cherry picked from commit b8bbbcf410dbcd287ab3ce9237e3934a88d4c5b4) --- eeschema/symbol_viewer_frame.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eeschema/symbol_viewer_frame.cpp b/eeschema/symbol_viewer_frame.cpp index fd289839e4..261094c1e5 100644 --- a/eeschema/symbol_viewer_frame.cpp +++ b/eeschema/symbol_viewer_frame.cpp @@ -1252,7 +1252,7 @@ void SYMBOL_VIEWER_FRAME::onSelectNextSymbol( wxCommandEvent& aEvent ) int ii = m_symbolList->GetSelection(); // Select the next symbol or stop at the end of the list. - if( ii != wxNOT_FOUND || ii != (int) m_symbolList->GetCount() - 1 ) + if( ii != wxNOT_FOUND && ii < (int)(m_symbolList->GetCount() - 1) ) ii += 1; m_symbolList->SetSelection( ii );