Fix recursive loops for flushing messages when more() is calling getch()

by adding a boolean to keep track on whether we already called
flush_prev_message() this round, reset once a new message is pushed into
prev_message. There's probably a better solution, but this one works
well enough.


git-svn-id: https://crawl-ref.svn.sourceforge.net/svnroot/crawl-ref/trunk@9285 c06c8d41-db1a-0410-9941-cceddc491573

1 files changed, 9 insertions, 2 deletions

diff --git a/crawl-ref/source/message.cc b/crawl-ref/source/message.cc
index 4e3f805b6f..e5165c3022 100644
--- a/crawl-ref/source/message.cc
+++ b/crawl-ref/source/message.cc
@@ -57,6 +57,8 @@ public:
 // Circular buffer for keeping past messages.
 message_item Store_Message[ NUM_STORED_MESSAGES ];    // buffer of old messages
 message_item prev_message;
+bool did_flush_message = false;
+
 int Next_Message = 0;                                 // end of messages
 
 int Message_Line = 0;                // line of next (previous?) message
@@ -757,9 +759,10 @@ static void handle_more(int colour)
 // something, e.g. answer a prompt.
 void flush_prev_message()
 {
-    if (prev_message.text.empty())
+    if (did_flush_message || prev_message.text.empty())
         return;
 
+    did_flush_message = true;
     base_mpr(prev_message.text.c_str(), prev_message.channel,
              prev_message.param, prev_message.colour, prev_message.repeats,
              false);
@@ -802,6 +805,7 @@ static void base_mpr(const char *inf, msg_channel_type channel, int param,
             prev_message.param   = param;
             prev_message.colour  = colour;
             prev_message.repeats = repeats;
+            did_flush_message = false;
             return;
         }
     }
@@ -845,7 +849,6 @@ static void base_mpr(const char *inf, msg_channel_type channel, int param,
 
 static void mpr_formatted_output(formatted_string fs, int colour)
 {
-    flush_prev_message();
     int curcol = Options.delay_message_clear ? 2 : 1;
 
     if (need_prefix)
@@ -886,6 +889,8 @@ static void mpr_formatted_output(formatted_string fs, int colour)
 void formatted_mpr(const formatted_string& fs, msg_channel_type channel,
                    int param)
 {
+    flush_prev_message();
+
     const std::string imsg = fs.tostring();
     const int colour = prepare_message(imsg, channel, param);
     if (colour == MSGCOL_MUTED)
@@ -914,6 +919,8 @@ void formatted_message_history(const std::string &st_nocolor,
                                msg_channel_type channel,
                                int param, int wrap_col)
 {
+    flush_prev_message();
+
     if (suppress_messages)
         return;