Changeset 099597c for fmtext.c

Timestamp:

Feb 4, 2011, 5:32:37 PM (15 years ago)

Author:

David Benjamin <davidben@mit.edu>

Branches:

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

Children:

8d553bf

Parents:

e062f97

git-author:

David Benjamin <davidben@mit.edu> (01/25/11 17:27:10)

git-committer:

David Benjamin <davidben@mit.edu> (02/04/11 17:32:37)

Message:

Don't reset colorpairs in the middle of drawing

Resetting colorpairs while drawing the mainwin causes the existing
contents in a popwin to refer to invalid color pairs. We used to draw
the mainwin first and redraw the contents of each window from scratch.
Moving to libpanel in 1.6 changed this, so background colors
occasionally bled into your popwin.

This changes the colorpair logic to only trigger when we need to, and to
forcibly dirty every window if needed. NOTE: if we don't have enough
color-pairs to draw the current screen, this will draw everything twice.
But it will probably almost never happen.

Reported-by: Alex Dehnert <adehnert@mit.edu>
Reviewed-by: Alejandro R. Sedeño <asedeno@mit.edu>

File:

• fmtext.c (modified) (3 diffs)

fmtext.c

@@ -876 +876 @@
   short i, j;
 
+  cpmgr->overflow = false;
   cpmgr->next = 8;
-    /* The test is <= because we allocated COLORS+1 entries. */
+  /* The test is <= because we allocated COLORS+1 entries. */
   for(i = 0; i <= COLORS; i++) {
     for(j = 0; j <= COLORS; j++) {
@@ -925 +926 @@
       /* We still don't have a pair, drop the background color. Too bad. */
       owl_function_debugmsg("colorpairs: color shortage - dropping background color.");
+      cpmgr->overflow = true;
       pair = owl_fmtext_get_colorpair(fg, OWL_COLOR_DEFAULT);
     }
@@ -930 +932 @@
       /* We still don't have a pair, defaults all around. */
       owl_function_debugmsg("colorpairs: color shortage - dropping foreground and background color.");
+      cpmgr->overflow = true;
       pair = 0;
     }