From: nethack.allison <nethack.allison>
Date: Mon, 13 Oct 2003 02:48:46 +0000 (+0000)
Subject: change the  "Stop eating?" question (trunk only)
X-Git-Tag: MOVE2GIT~1710
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2d99bb91be4438de48db082f8ea4f6e8c7bd1e68;p=nethack

change the  "Stop eating?" question (trunk only)

Opinions expressed:
> I actually like having "yes" consistently mean take a
> dangerous action, and "no" mean the safe route.

Also:
> I would classify the current behavior--where the "safe" answer
> is the opposite of the one for other prompts--as simply a bug.
[...]

change the dreaded "Stop eating?" prompt to "Continue eating?" with default "no"
---

diff --git a/doc/fixes35.0 b/doc/fixes35.0
index a5c65fa1c..e255ec76c 100644
--- a/doc/fixes35.0
+++ b/doc/fixes35.0
@@ -2,6 +2,7 @@ $RCSfile$ $Revision$ $Date$
 
 General Fixes and Modified Features
 -----------------------------------
+change the dreaded "Stop eating?" to "Continue eating?" with default "no"
 setmangry should not be called when a monster enters a bones region
 bad capitalization of msg when charging for something that catches light
 missing opthelp for use_inverse
diff --git a/src/eat.c b/src/eat.c
index 243144344..447608782 100644
--- a/src/eat.c
+++ b/src/eat.c
@@ -2189,7 +2189,8 @@ register int num;
 			context.victual.fullwarn = TRUE;
 			if (context.victual.canchoke && context.victual.reqtime > 1) {
 			    /* a one-gulp food will not survive a stop */
-			    if (yn_function("Stop eating?",ynchars,'y')=='y') {
+			    if (yn_function("Continue eating?",
+						ynchars,'n') != 'y') {
 				reset_eat();
 				nomovemsg = (char *)0;
 			    }