about summary refs log tree commit diff
diff options
context:
space:
mode:
authorPeter Stephenson <pws@users.sourceforge.net>2007-10-20 12:15:15 +0000
committerPeter Stephenson <pws@users.sourceforge.net>2007-10-20 12:15:15 +0000
commit4655eb4f9988b08e98aad7308d70a2fd91fa146c (patch)
tree78dad048b82868c0e2ff6011799d1426923d8be8
parent628291891275440ca60101a3cde29860f2a44161 (diff)
downloadzsh-4655eb4f9988b08e98aad7308d70a2fd91fa146c.tar.gz
zsh-4655eb4f9988b08e98aad7308d70a2fd91fa146c.tar.xz
zsh-4655eb4f9988b08e98aad7308d70a2fd91fa146c.zip
24001: fix up zcurses -C
-rw-r--r--ChangeLog4
-rw-r--r--Src/Modules/curses.c44
2 files changed, 29 insertions, 19 deletions
diff --git a/ChangeLog b/ChangeLog
index 816598e11..23034593b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2007-10-20  Peter Stephenson  <p.w.stephenson@ntlworld.com>
+
+	* 24001: Src/Modules/curses.c: fix up zcurses -C.
+
 2007-10-20  Clint Adams  <clint@zsh.org>
 
 	* Mikael Magnusson: 23999: Completion/Unix/Command/_git:
diff --git a/Src/Modules/curses.c b/Src/Modules/curses.c
index 6b2d0a4d4..d2b85157e 100644
--- a/Src/Modules/curses.c
+++ b/Src/Modules/curses.c
@@ -54,6 +54,12 @@ struct zcurses_namenumberpair {
     int number;
 };
 
+struct colorpairnode {
+    struct hashnode node;
+    short colorpair;
+};
+typedef struct colorpairnode *Colorpairnode;
+
 static WINDOW *win_zero;
 static struct ttyinfo saved_tty_state;
 static struct ttyinfo curses_tty_state;
@@ -70,7 +76,8 @@ static HashTable zcurses_colorpairs = NULL;
 #define ZCURSES_ATTRON 1
 #define ZCURSES_ATTROFF 2
 
-static int zc_errno, zc_color_phase=0, next_cp=0;
+static int zc_errno, zc_color_phase=0;
+static short next_cp=0;
 
 static const char *
 zcurses_strerror(int err)
@@ -172,7 +179,7 @@ zcurses_attribute(WINDOW *w, char *attr, int op)
     return 1;
 }
 
-static int
+static short
 zcurses_color(char *color)
 {
     struct zcurses_namenumberpair *zc;
@@ -191,19 +198,21 @@ zcurses_color(char *color)
 
     for(zc=(struct zcurses_namenumberpair *)zcurses_colors;zc->name;zc++)
 	if (!strcmp(color, zc->name)) {
-	    return zc->number;
+	    return (short)zc->number;
 	}
 
-    return -1;
+    return (short)-1;
 }
 
 static int
 zcurses_colorset(WINDOW *w, char *colorpair)
 {
     char *fg, *bg, *cp;
-    int *c, f, b;
+    short f, b;
+    Colorpairnode cpn;
 
-    if (zc_color_phase==1 || !(c = (int *) gethashnode(zcurses_colorpairs, colorpair))) {
+    if (zc_color_phase==1 ||
+	!(cpn = (Colorpairnode) gethashnode(zcurses_colorpairs, colorpair))) {
 	zc_color_phase = 2;
 	cp = ztrdup(colorpair);
 	fg = strtok(cp, "/");
@@ -229,26 +238,23 @@ zcurses_colorset(WINDOW *w, char *colorpair)
 	if (init_pair(next_cp, f, b) == ERR)
 	    return 1;
 
-	c = (int *)zalloc(sizeof(int *));
+	cpn = (Colorpairnode)zalloc(sizeof(struct colorpairnode));
 	
-	if(!c)
+	if (!cpn)
 	    return 1;
 
-	*c = next_cp;
-	addhashnode(zcurses_colorpairs, colorpair, (void *)c);
-    } 
-
-	fprintf(stderr, "%d\n", *c);
+	cpn->colorpair = next_cp;
+	addhashnode(zcurses_colorpairs, ztrdup(colorpair), (void *)cpn);
+    }
 
-    return (wcolor_set(w, *c, NULL) == ERR);
+    return (wcolor_set(w, cpn->colorpair, NULL) == ERR);
 }
 
 static void
-freecolornode(HashNode hn)
+freecolorpairnode(HashNode hn)
 {
-    int *i = (int *) hn;
-
-    zfree(i, sizeof(int));
+    zsfree(hn->nam);
+    zfree(hn, sizeof(struct colorpairnode));
 }
 
 /**/
@@ -275,7 +281,7 @@ bin_zcurses(char *nam, char **args, Options ops, UNUSED(int func))
 		zcurses_colorpairs->removenode  = removehashnode;
 		zcurses_colorpairs->disablenode = NULL;
 		zcurses_colorpairs->enablenode  = NULL;
-		zcurses_colorpairs->freenode    = freecolornode;
+		zcurses_colorpairs->freenode    = freecolorpairnode;
 		zcurses_colorpairs->printnode   = NULL;
 
 	    }