Subject: Unnecessary output from "update_titlebar_message"

From: Chuck Goodhart <[email-masked]>
Date: Tue, 1 Dec 1998 14:23:13 -0800 (PST)

The routine "update_titlebar_message" (lines 1289-1343 of screen.c in
pine4.05/pine) performs a "StartInverse/EndInverse" pair even if there
is no output between them.  This unnecessary output prevents my
demand-dialed link from going idle.  The following patch corrects the
problem:

--- pine4.05/pine/screen.c.orig	Wed Sep 23 11:56:43 1998
+++ pine4.05/pine/screen.c	Mon Nov 30 20:51:28 1998
@@ -1304,6 +1304,9 @@
     if(as.cur_mess_col < 0)
       return;
 
+    if(mn_get_cur(as.msgmap) == as.current_msg)
+      return;
+
     delta = digit_count(mn_get_cur(as.msgmap)) - digit_count(as.current_msg);
 
     StartInverse();
@@ -1330,7 +1333,7 @@
 	    as.del_column += delta;
         }
     }
-    else if(mn_get_cur(as.msgmap) != as.current_msg){
+    else {
 	as.current_msg = mn_get_cur(as.msgmap);
 	PutLine0(0, as.cur_mess_col, comatose(as.current_msg));
     }

-- 
Chuck Goodhart <[email-masked]>

-----------------------------------------------------------------
 For information about this mailing list, and its archives, see: 
 http://www.washington.edu/pine/pine-info/
-----------------------------------------------------------------