]> granicus.if.org Git - nethack/commitdiff
fixes entry for pull request #993 - curses' >>
authorPatR <rankin@nethack.org>
Fri, 3 Mar 2023 19:16:13 +0000 (11:16 -0800)
committerPatR <rankin@nethack.org>
Fri, 3 Mar 2023 19:16:13 +0000 (11:16 -0800)
Pull request from entrez:  when the curses interface was deciding
whether ">>" could fit on the bottom line of the message window, it
was off by two (based on the initial report) and ">>" clobbered the
last two characters.

Reported case was "Welcome to level " when that was appended to
another short message.  It should have ended with "6. " but that had
been chopped off by ">> ".  Reproducible by assigning a long name to
something, dropping an item, dropping the something, and dropping
just enough other stuff so that the named item will be reported on
the bottom message line during pickup-all, forcing --More-- before
continuing to the item below it.  The item with the long name needs
to fit in the message window's width (when formatted with invent
letter prefix and trailing period) without being wrapped and not
leaving enough unused space to fix ">> ".

Closes #993

doc/fixes3-7-0.txt

index 0bbd643f1725ecf0c207171c1698ab584587bf69..ceada7cf01cde0b1d62b29ffc79e16878ff12ac6 100644 (file)
@@ -1601,6 +1601,8 @@ curses: implement mark_synch() and wait_synch()
 curses: only convert arrow keys to hjkl or 4286 if nethack wants a direction
 curses: conversion of arrows and other numpad-related function keys to digit
        ignored phone-layout setting (inverted up/down) of the num_pad option
+curses: fix an off-by-one error when deciding whether a long line on the
+       bottom line of the message window can fit ">>" (curses' "--More--")
 macOS: Xcode project was failing to build if the path to the NetHack source
        tree contained a space; the issue was within some shell script code
        contained within the project