]> granicus.if.org Git - nethack/commitdiff
fixes entry for pull request #881 - curses exit
authorPatR <rankin@nethack.org>
Wed, 28 Sep 2022 21:31:11 +0000 (14:31 -0700)
committerPatR <rankin@nethack.org>
Wed, 28 Sep 2022 21:31:11 +0000 (14:31 -0700)
Pull request from chasonr:  if the curses interface did a full screen
update while the game was ending, it could attempt to refresh windows
that had already been removed.

For #if PDCURSES, raw_print() could attempt to write to the message
window after it had been removed.

[Why is raw_print() using the message window at all?  Isn't the main
point of it to bypass that?]

Fixes #881

doc/fixes3-7-0.txt

index bc8b29281f74879f223eb371aca051a512fdedc1..c50e133693274efca9a9a7d6d0217be2ffe1f038 100644 (file)
@@ -1420,6 +1420,10 @@ curses: clipped map crashed due to uninitialized memory
 curses: changes for tty's TTY_PERM_INVENT resulted in viewing inventory under
        curses while carrying only gold to crash; didn't matter if perm_invent
        was On or if TTY_PERM_INVENT was enabled
+curses: for screen refresh, don't attempt to refresh windows that haven't been
+       set up yet or have already been shut down
+curses: for raw_print() in #if PDCURSES config, don't attempt to write to
+       message window if it isn't set up yet or has already been shut down
 Qt: at Xp levels above 20 with 'showexp' On, the combined status field
        "Level:NN/nnnnnnnn" was too big and truncated by a char at each end
 Qt: searching a text window for something that wasn't found and then searching