Changeset 5eeea3b for functions.c

Timestamp:

Aug 17, 2002, 3:23:00 PM (22 years ago)

Author:

Erik Nygren <nygren@mit.edu>

Branches:

master, barnowl_perlaim, debian, owl, release-1.10, release-1.4, release-1.5, release-1.6, release-1.7, release-1.8, release-1.9

Children:

1354456

Parents:

9bd2c17

Message:

	Modified the behavior of last so that "> >" will clear the screen.
	The new behavior of last is:
	      Moves the pointer to the last message in the view.
	      If we are already at the last message in the view,
	      blanks the screen and moves just past the end of the view
	      so that new messages will appear starting at the top
	      of the screen.
	Fixed a typo in the help for smartzpunt.
	Fixed functions to handle curmsg being past the end of the view.

File:

• functions.c (modified) (10 diffs)

functions.c

@@ -375 +375 @@
   v=owl_global_get_current_view(&g);
   ml=owl_global_get_msglist(&g);
+
+  /* just check to make sure we're in bounds... */
+  if (curmsg>owl_view_get_size(v)-1) curmsg=owl_view_get_size(v)-1;
+  if (curmsg<0) curmsg=0;
 
   /* first try to move to an undeleted message in the view*/
@@ -389 +393 @@
 
     /* if we weren't successful try to find one backwards */
-    curmsg=owl_global_get_curmsg(&g);
     if (owl_message_is_delete(owl_view_get_element(v, curmsg))) {
       for (i=curmsg; i>0; i--) {
@@ -431 +434 @@
 
 void owl_function_lastmsg_noredisplay() {
-  int curmsg;
+  int oldcurmsg, curmsg;
   owl_view *v;
 
   v=owl_global_get_current_view(&g);
-  
-  curmsg=owl_view_get_size(v)-1;
+  oldcurmsg=owl_global_get_curmsg(&g);
+  curmsg=owl_view_get_size(v)-1;  
   if (curmsg<0) curmsg=0;
   owl_global_set_curmsg(&g, curmsg);
-  owl_function_calculate_topmsg(OWL_DIRECTION_DOWNWARDS);
+  if (oldcurmsg < curmsg) {
+    owl_function_calculate_topmsg(OWL_DIRECTION_DOWNWARDS);
+  } else if (curmsg<owl_view_get_size(v)) {
+    /* If already at the end, blank the screen and move curmsg
+     * past the end of the messages. */
+    owl_global_set_topmsg(&g, curmsg+1);
+    owl_global_set_curmsg(&g, curmsg+1);
+  } 
   owl_mainwin_redisplay(owl_global_get_mainwin(&g));
   owl_global_set_direction_downwards(&g);
@@ -631 +641 @@
   v=owl_global_get_current_view(&g);
   
-  if (owl_view_get_size(v)==0) {
+  m=owl_view_get_element(v, owl_global_get_curmsg(&g));
+
+  if (!m || owl_view_get_size(v)==0) {
     owl_function_makemsg("No current message selected");
     return;
   }
 
-  m=owl_view_get_element(v, owl_global_get_curmsg(&g));
   text=owl_message_get_text(m);
 
@@ -1056 +1067 @@
   v=owl_global_get_current_view(&g);
   
-  if (owl_view_get_size(v)==0) {
+  m=owl_view_get_element(v, owl_global_get_curmsg(&g));
+
+  if (!m || owl_view_get_size(v)==0) {
     owl_function_makemsg("No message selected\n");
     return;
   }
 
-  m=owl_view_get_element(v, owl_global_get_curmsg(&g));
   if (!owl_message_is_zephyr(m)) {
     sprintf(buff,   "Owl Message Id: %i\n", owl_message_get_id(m));
@@ -1158 +1170 @@
   pw=owl_global_get_popwin(&g);
 
-  if (owl_view_get_size(v)==0) {
+  m=owl_view_get_element(v, owl_global_get_curmsg(&g));
+
+  if (!m || owl_view_get_size(v)==0) {
     owl_function_makemsg("No current message");
     return;
   }
 
-  m=owl_view_get_element(v, owl_global_get_curmsg(&g));
   owl_function_popless_fmtext(owl_message_get_fmtext(m));
 }
@@ -1177 +1190 @@
   offset=owl_global_get_curmsg_vert_offset(&g);
   v=owl_global_get_current_view(&g);
-  if (owl_view_get_size(v)==0) return;
   curmsg=owl_global_get_curmsg(&g);
   m=owl_view_get_element(v, curmsg);
+  if (!m || owl_view_get_size(v)==0) return;
   lines=owl_message_get_numlines(m);
 
@@ -1479 +1492 @@
     
     m=owl_view_get_element(owl_global_get_current_view(&g), owl_global_get_curmsg(&g));
+    if (!m) {
+      owl_function_makemsg("No message selected");
+      return;
+    }
+
 
     /* first check if we catch the reply-lockout filter */
@@ -1986 +2004 @@
   m=owl_view_get_element(v, owl_global_get_curmsg(&g));
 
-  if (owl_view_get_size(v)==0) {
+  if (!m || owl_view_get_size(v)==0) {
     owl_function_makemsg("No message selected\n");
     return NULL;
@@ -2033 +2051 @@
   m=owl_view_get_element(v, owl_global_get_curmsg(&g));
 
-  if (owl_view_get_size(v)==0) {
+  if (!m || owl_view_get_size(v)==0) {
     owl_function_makemsg("No message selected\n");
     return;