Changeset 60c1386 for viewwin.c

Timestamp:

Sep 18, 2010, 5:07:39 PM (14 years ago)

Author:

David Benjamin <davidben@mit.edu>

Branches:

master, release-1.10, release-1.7, release-1.8, release-1.9

Children:

a8186dc

Parents:

2640b63

git-author:

David Benjamin <davidben@mit.edu> (08/01/10 16:29:45)

git-committer:

David Benjamin <davidben@mit.edu> (09/18/10 17:07:39)

Message:

Split the viewwin's status bar into its own window

File:

• viewwin.c (modified) (12 diffs)

viewwin.c

@@ -5 +5 @@
 #define EMPTY_INDICATOR "~"
 
-static void owl_viewwin_redraw(owl_window *w, WINDOW *curswin, void *user_data);
+static void owl_viewwin_redraw_content(owl_window *w, WINDOW *curswin, void *user_data);
+static void owl_viewwin_redraw_status(owl_window *w, WINDOW *curswin, void *user_data);
+static void owl_viewwin_layout(owl_viewwin *v);
 static void owl_viewwin_set_window(owl_viewwin *v, owl_window *w);
 
@@ -56 +58 @@
 static void owl_viewwin_set_window(owl_viewwin *v, owl_window *w)
 {
-  v->window = w;
-  if (w) {
-    g_object_ref(v->window);
-    v->sig_redraw_id = g_signal_connect(w, "redraw", G_CALLBACK(owl_viewwin_redraw), v);
-  }
+  v->window = g_object_ref(w);
+  v->content = owl_window_new(v->window);
+  v->status = owl_window_new(v->window);
+
+  v->sig_content_redraw_id =
+    g_signal_connect(v->content, "redraw", G_CALLBACK(owl_viewwin_redraw_content), v);
+  v->sig_status_redraw_id =
+    g_signal_connect(v->status, "redraw", G_CALLBACK(owl_viewwin_redraw_status), v);
+  v->sig_resize_id =
+    g_signal_connect_swapped(v->window, "resized", G_CALLBACK(owl_viewwin_layout), v);
+  owl_viewwin_layout(v);
+
+  owl_window_show(v->content);
+  owl_window_show(v->status);
 }
 
@@ -68 +79 @@
 }
 
+static void owl_viewwin_layout(owl_viewwin *v)
+{
+  int lines, cols;
+  owl_window_get_position(v->window, &lines, &cols, NULL, NULL);
+  owl_window_set_position(v->content, lines - BOTTOM_OFFSET, cols, 0, 0);
+  owl_window_set_position(v->status, BOTTOM_OFFSET, cols, lines - BOTTOM_OFFSET, 0);
+}
+
 /* regenerate text on the curses window. */
-static void owl_viewwin_redraw(owl_window *w, WINDOW *curswin, void *user_data)
+static void owl_viewwin_redraw_content(owl_window *w, WINDOW *curswin, void *user_data)
 {
   owl_fmtext fm1, fm2;
@@ -77 +96 @@
 
   owl_window_get_position(w, &winlines, &wincols, 0, 0);
-  
+
   werase(curswin);
   wmove(curswin, 0, 0);
@@ -83 +102 @@
   owl_fmtext_init_null(&fm1);
   owl_fmtext_init_null(&fm2);
-  
-  owl_fmtext_truncate_lines(&(v->fmtext), v->topline, winlines-BOTTOM_OFFSET, &fm1);
+
+  owl_fmtext_truncate_lines(&(v->fmtext), v->topline, winlines, &fm1);
   owl_fmtext_truncate_cols(&fm1, v->rightshift, wincols-1+v->rightshift, &fm2);
 
@@ -92 +111 @@
   y = v->textlines - v->topline;
   wmove(curswin, y, 0);
-  for (; y < winlines-BOTTOM_OFFSET; y++) {
+  for (; y < winlines; y++) {
     waddstr(curswin, EMPTY_INDICATOR);
     waddstr(curswin, "\n");
   }
 
-  /* print the message at the bottom */
-  wmove(curswin, winlines-1, 0);
+  owl_fmtext_cleanup(&fm1);
+  owl_fmtext_cleanup(&fm2);
+}
+
+static void owl_viewwin_redraw_status(owl_window *w, WINDOW *curswin, void *user_data)
+{
+  owl_viewwin *v = user_data;
+  int winlines, wincols;
+
+  owl_window_get_position(v->content, &winlines, &wincols, 0, 0);
+
+  werase(curswin);
+  wmove(curswin, 0, 0);
   wattrset(curswin, A_REVERSE);
-  if (v->textlines - v->topline > winlines-BOTTOM_OFFSET) {
+  if (v->textlines - v->topline > winlines) {
     waddstr(curswin, "--More-- (Space to see more, 'q' to quit)");
   } else {
@@ -106 +136 @@
   }
   wattroff(curswin, A_REVERSE);
-
-  owl_fmtext_cleanup(&fm1);
-  owl_fmtext_cleanup(&fm2);
 }
 
@@ -122 +149 @@
 void owl_viewwin_dirty(owl_viewwin *v)
 {
-  if (v->window)
-    owl_window_dirty(v->window);
+  owl_window_dirty(v->content);
+  owl_window_dirty(v->status);
 }
 
 void owl_viewwin_down(owl_viewwin *v, int amount) {
   int winlines;
-  owl_window_get_position(v->window, &winlines, 0, 0, 0);
+  owl_window_get_position(v->content, &winlines, 0, 0, 0);
   /* Don't scroll past the bottom. */
-  amount = MIN(amount, v->textlines - (v->topline + winlines - BOTTOM_OFFSET));
+  amount = MIN(amount, v->textlines - (v->topline + winlines));
   /* But if we're already past the bottom, don't back up either. */
   if (amount > 0) {
@@ -148 +175 @@
 {
   int winlines;
-  owl_window_get_position(v->window, &winlines, 0, 0, 0);
-  owl_viewwin_down(v, winlines - BOTTOM_OFFSET);
+  owl_window_get_position(v->content, &winlines, 0, 0, 0);
+  owl_viewwin_down(v, winlines);
 }
 
@@ -160 +187 @@
 {
   int winlines;
-  owl_window_get_position(v->window, &winlines, 0, 0, 0);
-  owl_viewwin_up(v, winlines - BOTTOM_OFFSET);
+  owl_window_get_position(v->content, &winlines, 0, 0, 0);
+  owl_viewwin_up(v, winlines);
 }
 
@@ -192 +219 @@
 {
   int winlines;
-  owl_window_get_position(v->window, &winlines, 0, 0, 0);
-  v->topline = v->textlines - winlines + BOTTOM_OFFSET;
+  owl_window_get_position(v->content, &winlines, 0, 0, 0);
+  v->topline = v->textlines - winlines;
   owl_viewwin_dirty(v);
 }
@@ -228 +255 @@
     v->onclose_hook_data = NULL;
   }
-  if (v->window) {
-    g_signal_handler_disconnect(v->window, v->sig_redraw_id);
-    g_object_unref(v->window);
-    v->window = NULL;
-  }
+  /* TODO: This is far too tedious. owl_viewwin should own v->window
+   * and just unlink it in one go. Signals should also go away for
+   * free. */
+  g_signal_handler_disconnect(v->window, v->sig_resize_id);
+  g_signal_handler_disconnect(v->content, v->sig_content_redraw_id);
+  g_signal_handler_disconnect(v->status, v->sig_status_redraw_id);
+  owl_window_unlink(v->content);
+  owl_window_unlink(v->status);
+  g_object_unref(v->window);
+  g_object_unref(v->content);
+  g_object_unref(v->status);
   owl_fmtext_cleanup(&(v->fmtext));
   owl_free(v);