xenia/kakoune
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix NCursesClient::menu_select for one past last item

Browse Source
This commit is contained in:
Maxime Coste 2012-09-11 14:28:54 +02:00
parent ee4c6b04a6
commit 859e907396
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/ncurses.cc
View File

@ -254,7 +254,8 @@ void NCursesClient::menu_show(const memoryview<String>& choices)
void NCursesClient::menu_select(int selected) void NCursesClient::menu_select(int selected)
{ {
if (0 <= selected and selected < m_items.size()) // last item in m_items is the nullptr, hence the - 1
if (selected >= 0 and selected < m_items.size() - 1)
{ {
set_menu_fore(m_menu, COLOR_PAIR(m_menu_fg)); set_menu_fore(m_menu, COLOR_PAIR(m_menu_fg));
set_current_item(m_menu, m_items[selected]); set_current_item(m_menu, m_items[selected]);