]> granicus.if.org Git - nethack/commitdiff
Qt text search crash fix
authorPatR <rankin@nethack.org>
Sat, 12 Dec 2020 02:11:27 +0000 (18:11 -0800)
committerPatR <rankin@nethack.org>
Sat, 12 Dec 2020 02:11:27 +0000 (18:11 -0800)
About 5 weeks ago, commit e4106bb1613da808ae87780dfeedb4c80e6afc2a
changed Qt's searching in text windows to be able to find a match
on the very first line.  It assumed that the first line would be
the current line except when repeating the same search after a
match.  But after a failed search the current line index is -1 and
starting a new search would crash trying to look that line up.

win/Qt/qt_menu.cpp

index ab8b59ea75a3feaeb816de17267ef0305045d8d1..1a549a8e04aa3c89e7c2293f72c59ecd3e6a6e1c 100644 (file)
@@ -1192,6 +1192,8 @@ void NetHackQtTextWindow::Search()
     if (get_a_line) {
         int linecount = lines->count();
         int current = lines->currentRow();
+        if (current == -1)
+            current = 0;
         // when no row is highlighted (selected), start the search
         // on the current row, otherwise start on the row after it
         // [normally means that the very first row is a candidate