]> ncurses.scripts.mit.edu Git - ncurses.git/blobdiff - ncurses/base/lib_color.c
ncurses 5.6 - patch 20070421
[ncurses.git] / ncurses / base / lib_color.c
index 71bee42487cd9c19ae0cfe102b92af1aba4f8a78..9cae495436fecc4f3a3b80546e6a18d05fe4dbec 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1998,1999,2000 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2006,2007 Free Software Foundation, Inc.              *
  *                                                                          *
  * Permission is hereby granted, free of charge, to any person obtaining a  *
  * copy of this software and associated documentation files (the            *
@@ -29,6 +29,7 @@
 /****************************************************************************
  *  Author: Zeyd M. Ben-Halim <zmbenhal@netcom.com> 1992,1995               *
  *     and: Eric S. Raymond <esr@snark.thyrsus.com>                         *
+ *     and: Thomas E. Dickey                        1996-on                 *
  ****************************************************************************/
 
 /* lib_color.c
 #include <term.h>
 #include <tic.h>
 
-MODULE_ID("$Id: lib_color.c,v 1.51 2000/05/20 20:09:22 tom Exp $")
+MODULE_ID("$Id: lib_color.c,v 1.85 2007/04/07 17:07:28 tom Exp $")
 
 /*
  * These should be screen structure members.  They need to be globals for
  * historical reasons.  So we assign them in start_color() and also in
  * set_term()'s screen-switching logic.
  */
-int COLOR_PAIRS = 0;
-int COLORS = 0;
+#if USE_REENTRANT
+NCURSES_EXPORT(int)
+NCURSES_PUBLIC_VAR(COLOR_PAIRS) (void)
+{
+    return SP ? SP->_pair_count : -1;
+}
+NCURSES_EXPORT(int)
+NCURSES_PUBLIC_VAR(COLORS) (void)
+{
+    return SP ? SP->_color_count : -1;
+}
+#else
+NCURSES_EXPORT_VAR(int) COLOR_PAIRS = 0;
+NCURSES_EXPORT_VAR(int) COLORS = 0;
+#endif
+
+#define DATA(r,g,b) {r,g,b, 0,0,0, 0}
+
+#define TYPE_CALLOC(type,elts) typeCalloc(type, (unsigned)(elts))
+
+#define MAX_PALETTE    8
+
+#define OkColorHi(n)   (((n) < COLORS) && ((n) < max_colors))
+#define InPalette(n)   ((n) >= 0 && (n) < MAX_PALETTE)
 
 /*
  * Given a RGB range of 0..1000, we'll normally set the individual values
@@ -61,31 +84,31 @@ int COLORS = 0;
 static const color_t cga_palette[] =
 {
     /*  R               G               B */
-    {RGB_OFF,          RGB_OFF,        RGB_OFF},       /* COLOR_BLACK */
-    {RGB_ON,           RGB_OFF,        RGB_OFF},       /* COLOR_RED */
-    {RGB_OFF,          RGB_ON,         RGB_OFF},       /* COLOR_GREEN */
-    {RGB_ON,           RGB_ON,         RGB_OFF},       /* COLOR_YELLOW */
-    {RGB_OFF,          RGB_OFF,        RGB_ON},        /* COLOR_BLUE */
-    {RGB_ON,           RGB_OFF,        RGB_ON},        /* COLOR_MAGENTA */
-    {RGB_OFF,          RGB_ON,         RGB_ON},        /* COLOR_CYAN */
-    {RGB_ON,           RGB_ON,         RGB_ON},        /* COLOR_WHITE */
+    DATA(RGB_OFF,      RGB_OFF,        RGB_OFF),       /* COLOR_BLACK */
+    DATA(RGB_ON,       RGB_OFF,        RGB_OFF),       /* COLOR_RED */
+    DATA(RGB_OFF,      RGB_ON,         RGB_OFF),       /* COLOR_GREEN */
+    DATA(RGB_ON,       RGB_ON,         RGB_OFF),       /* COLOR_YELLOW */
+    DATA(RGB_OFF,      RGB_OFF,        RGB_ON),        /* COLOR_BLUE */
+    DATA(RGB_ON,       RGB_OFF,        RGB_ON),        /* COLOR_MAGENTA */
+    DATA(RGB_OFF,      RGB_ON,         RGB_ON),        /* COLOR_CYAN */
+    DATA(RGB_ON,       RGB_ON,         RGB_ON),        /* COLOR_WHITE */
 };
 
 static const color_t hls_palette[] =
 {
-    /*  H       L       S */
-    {  0,      0,      0},             /* COLOR_BLACK */
-    {  120,    50,     100},           /* COLOR_RED */
-    {  240,    50,     100},           /* COLOR_GREEN */
-    {  180,    50,     100},           /* COLOR_YELLOW */
-    {  330,    50,     100},           /* COLOR_BLUE */
-    {  60,     50,     100},           /* COLOR_MAGENTA */
-    {  300,    50,     100},           /* COLOR_CYAN */
-    {  0,      50,     100},           /* COLOR_WHITE */
+    /*         H       L       S */
+    DATA(      0,      0,      0),             /* COLOR_BLACK */
+    DATA(      120,    50,     100),           /* COLOR_RED */
+    DATA(      240,    50,     100),           /* COLOR_GREEN */
+    DATA(      180,    50,     100),           /* COLOR_YELLOW */
+    DATA(      330,    50,     100),           /* COLOR_BLUE */
+    DATA(      60,     50,     100),           /* COLOR_MAGENTA */
+    DATA(      300,    50,     100),           /* COLOR_CYAN */
+    DATA(      0,      50,     100),           /* COLOR_WHITE */
 };
 /* *INDENT-ON* */
 
-#ifdef NCURSES_EXT_FUNCS
+#if NCURSES_EXT_FUNCS
 /*
  * These are called from _nc_do_color(), which in turn is called from
  * vidattr - so we have to assume that SP may be null.
@@ -117,7 +140,7 @@ toggled_colors(int c)
     if (c < 16) {
        static const int table[] =
        {0, 4, 2, 6, 1, 5, 3, 7,
-           8, 12, 10, 14, 9, 13, 11, 15};
+        8, 12, 10, 14, 9, 13, 11, 15};
        c = table[c];
     }
     return c;
@@ -128,10 +151,10 @@ set_background_color(int bg, int (*outc) (int))
 {
     if (set_a_background) {
        TPUTS_TRACE("set_a_background");
-       tputs(tparm(set_a_background, bg), 1, outc);
+       tputs(TPARM_1(set_a_background, bg), 1, outc);
     } else {
        TPUTS_TRACE("set_background");
-       tputs(tparm(set_background, toggled_colors(bg)), 1, outc);
+       tputs(TPARM_1(set_background, toggled_colors(bg)), 1, outc);
     }
 }
 
@@ -140,62 +163,25 @@ set_foreground_color(int fg, int (*outc) (int))
 {
     if (set_a_foreground) {
        TPUTS_TRACE("set_a_foreground");
-       tputs(tparm(set_a_foreground, fg), 1, outc);
+       tputs(TPARM_1(set_a_foreground, fg), 1, outc);
     } else {
        TPUTS_TRACE("set_foreground");
-       tputs(tparm(set_foreground, toggled_colors(fg)), 1, outc);
-    }
-}
-
-static bool
-set_original_colors(void)
-{
-    if (orig_pair != 0) {
-       TPUTS_TRACE("orig_pair");
-       putp(orig_pair);
-       return TRUE;
-    } else if (orig_colors != NULL) {
-       TPUTS_TRACE("orig_colors");
-       putp(orig_colors);
-       return TRUE;
+       tputs(TPARM_1(set_foreground, toggled_colors(fg)), 1, outc);
     }
-    return FALSE;
 }
 
-int
-start_color(void)
+static void
+init_color_table(void)
 {
-    int n;
     const color_t *tp;
+    int n;
 
-    T((T_CALLED("start_color()")));
-
-    if (set_original_colors() != TRUE) {
-       set_foreground_color(default_fg(), _nc_outch);
-       set_background_color(default_bg(), _nc_outch);
-    }
-
-    if (VALID_NUMERIC(max_pairs))
-       COLOR_PAIRS = SP->_pair_count = max_pairs;
-    else
-       returnCode(ERR);
-    if ((SP->_color_pairs = typeCalloc(unsigned short, max_pairs)) == 0)
-         returnCode(ERR);
-    SP->_color_pairs[0] = PAIR_OF(default_fg(), default_bg());
-    if (VALID_NUMERIC(max_colors))
-       COLORS = SP->_color_count = max_colors;
-    else
-       returnCode(ERR);
-    SP->_coloron = 1;
-
-    if ((SP->_color_table = typeMalloc(color_t, COLORS)) == 0)
-       returnCode(ERR);
     tp = (hue_lightness_saturation) ? hls_palette : cga_palette;
     for (n = 0; n < COLORS; n++) {
-       if (n < 8) {
+       if (InPalette(n)) {
            SP->_color_table[n] = tp[n];
        } else {
-           SP->_color_table[n] = tp[n % 8];
+           SP->_color_table[n] = tp[n % MAX_PALETTE];
            if (hue_lightness_saturation) {
                SP->_color_table[n].green = 100;
            } else {
@@ -208,10 +194,95 @@ start_color(void)
            }
        }
     }
+}
 
-    T(("started color: COLORS = %d, COLOR_PAIRS = %d", COLORS, COLOR_PAIRS));
+/*
+ * Reset the color pair, e.g., to whatever color pair 0 is.
+ */
+static bool
+reset_color_pair(void)
+{
+    bool result = FALSE;
 
-    returnCode(OK);
+    if (orig_pair != 0) {
+       TPUTS_TRACE("orig_pair");
+       putp(orig_pair);
+       result = TRUE;
+    }
+    return result;
+}
+
+/*
+ * Reset color pairs and definitions.  Actually we do both more to accommodate
+ * badly-written terminal descriptions than for the relatively rare case where
+ * someone has changed the color definitions.
+ */
+bool
+_nc_reset_colors(void)
+{
+    int result = FALSE;
+
+    T((T_CALLED("_nc_reset_colors()")));
+    if (SP->_color_defs > 0)
+       SP->_color_defs = -(SP->_color_defs);
+
+    if (reset_color_pair())
+       result = TRUE;
+    if (orig_colors != 0) {
+       TPUTS_TRACE("orig_colors");
+       putp(orig_colors);
+       result = TRUE;
+    }
+    returnBool(result);
+}
+
+NCURSES_EXPORT(int)
+start_color(void)
+{
+    int result = ERR;
+
+    T((T_CALLED("start_color()")));
+
+    if (SP == 0) {
+       result = ERR;
+    } else if (SP->_coloron) {
+       result = OK;
+    } else {
+
+       if (reset_color_pair() != TRUE) {
+           set_foreground_color(default_fg(), _nc_outch);
+           set_background_color(default_bg(), _nc_outch);
+       }
+
+       if (max_pairs > 0 && max_colors > 0) {
+           SP->_pair_count = max_pairs;
+           SP->_color_count = max_colors;
+#if !USE_REENTRANT
+           COLOR_PAIRS = max_pairs;
+           COLORS = max_colors;
+#endif
+
+           if ((SP->_color_pairs = TYPE_CALLOC(colorpair_t,
+                                               max_pairs)) != 0) {
+               if ((SP->_color_table = TYPE_CALLOC(color_t,
+                                                   max_colors)) != 0) {
+                   SP->_color_pairs[0] = PAIR_OF(default_fg(), default_bg());
+                   init_color_table();
+
+                   T(("started color: COLORS = %d, COLOR_PAIRS = %d",
+                      COLORS, COLOR_PAIRS));
+
+                   SP->_coloron = 1;
+                   result = OK;
+               } else if (SP->_color_pairs != 0) {
+                   FreeAndNull(SP->_color_pairs);
+               }
+           }
+       } else {
+           result = OK;
+       }
+    }
+    returnCode(result);
 }
 
 /* This function was originally written by Daniel Weaver <danw@znyx.com> */
@@ -256,30 +327,30 @@ rgb2hls(short r, short g, short b, short *h, short *l, short *s)
  * Extension (1997/1/18) - Allow negative f/b values to set default color
  * values.
  */
-int
+NCURSES_EXPORT(int)
 init_pair(short pair, short f, short b)
 {
-    unsigned result;
+    colorpair_t result;
 
     T((T_CALLED("init_pair(%d,%d,%d)"), pair, f, b));
 
-    if ((pair < 0) || (pair >= COLOR_PAIRS))
+    if ((pair < 0) || (pair >= COLOR_PAIRS) || SP == 0 || !SP->_coloron)
        returnCode(ERR);
-#ifdef NCURSES_EXT_FUNCS
+#if NCURSES_EXT_FUNCS
     if (SP->_default_color) {
        if (f < 0)
-           f = C_MASK;
+           f = COLOR_DEFAULT;
        if (b < 0)
-           b = C_MASK;
-       if (f >= COLORS && f != C_MASK)
+           b = COLOR_DEFAULT;
+       if (!OkColorHi(f) && !isDefaultColor(f))
            returnCode(ERR);
-       if (b >= COLORS && b != C_MASK)
+       if (!OkColorHi(b) && !isDefaultColor(b))
            returnCode(ERR);
     } else
 #endif
     {
-       if ((f < 0) || (f >= COLORS)
-           || (b < 0) || (b >= COLORS)
+       if ((f < 0) || !OkColorHi(f)
+           || (b < 0) || !OkColorHi(b)
            || (pair < 1))
            returnCode(ERR);
     }
@@ -293,16 +364,15 @@ init_pair(short pair, short f, short b)
     if (SP->_color_pairs[pair] != 0
        && SP->_color_pairs[pair] != result) {
        int y, x;
-       attr_t z = COLOR_PAIR(pair);
 
        for (y = 0; y <= curscr->_maxy; y++) {
            struct ldat *ptr = &(curscr->_line[y]);
            bool changed = FALSE;
            for (x = 0; x <= curscr->_maxx; x++) {
-               if ((ptr->text[x] & A_COLOR) == z) {
+               if (GetPair(ptr->text[x]) == pair) {
                    /* Set the old cell to zero to ensure it will be
                       updated on the next doupdate() */
-                   ptr->text[x] = 0;
+                   SetChar(ptr->text[x], 0, 0);
                    CHANGED_CELL(ptr, x);
                    changed = TRUE;
                }
@@ -312,114 +382,149 @@ init_pair(short pair, short f, short b)
        }
     }
     SP->_color_pairs[pair] = result;
-    if ((int) (SP->_current_attr & A_COLOR) == COLOR_PAIR(pair))
-       SP->_current_attr |= A_COLOR;   /* force attribute update */
+    if (GET_SCREEN_PAIR(SP) == pair)
+       SET_SCREEN_PAIR(SP, (chtype) (~0));     /* force attribute update */
 
-    if (initialize_pair) {
+    if (initialize_pair && InPalette(f) && InPalette(b)) {
        const color_t *tp = hue_lightness_saturation ? hls_palette : cga_palette;
 
-       T(("initializing pair: pair = %d, fg=(%d,%d,%d), bg=(%d,%d,%d)",
-               pair,
-               tp[f].red, tp[f].green, tp[f].blue,
-               tp[b].red, tp[b].green, tp[b].blue));
-
-       if (initialize_pair) {
-           TPUTS_TRACE("initialize_pair");
-           putp(tparm(initialize_pair,
-                   pair,
-                   tp[f].red, tp[f].green, tp[f].blue,
-                   tp[b].red, tp[b].green, tp[b].blue));
-       }
+       TR(TRACE_ATTRS,
+          ("initializing pair: pair = %d, fg=(%d,%d,%d), bg=(%d,%d,%d)",
+           pair,
+           tp[f].red, tp[f].green, tp[f].blue,
+           tp[b].red, tp[b].green, tp[b].blue));
+
+       TPUTS_TRACE("initialize_pair");
+       putp(TPARM_7(initialize_pair,
+                    pair,
+                    tp[f].red, tp[f].green, tp[f].blue,
+                    tp[b].red, tp[b].green, tp[b].blue));
     }
 
     returnCode(OK);
 }
 
-int
+#define okRGB(n) ((n) >= 0 && (n) <= 1000)
+
+NCURSES_EXPORT(int)
 init_color(short color, short r, short g, short b)
 {
-    T((T_CALLED("init_color(%d,%d,%d,%d)"), color, r, g, b));
-
-    if (initialize_color == NULL)
-       returnCode(ERR);
+    int result = ERR;
 
-    if (color < 0 || color >= COLORS)
-       returnCode(ERR);
-    if (r < 0 || r > 1000 || g < 0 || g > 1000 || b < 0 || b > 1000)
-       returnCode(ERR);
+    T((T_CALLED("init_color(%d,%d,%d,%d)"), color, r, g, b));
 
-    if (hue_lightness_saturation)
-       rgb2hls(r, g, b,
-           &SP->_color_table[color].red,
-           &SP->_color_table[color].green,
-           &SP->_color_table[color].blue);
-    else {
-       SP->_color_table[color].red = r;
-       SP->_color_table[color].green = g;
-       SP->_color_table[color].blue = b;
-    }
+    if (initialize_color != NULL
+       && SP != 0
+       && SP->_coloron
+       && (color >= 0 && OkColorHi(color))
+       && (okRGB(r) && okRGB(g) && okRGB(b))) {
+
+       SP->_color_table[color].init = 1;
+       SP->_color_table[color].r = r;
+       SP->_color_table[color].g = g;
+       SP->_color_table[color].b = b;
+
+       if (hue_lightness_saturation) {
+           rgb2hls(r, g, b,
+                   &SP->_color_table[color].red,
+                   &SP->_color_table[color].green,
+                   &SP->_color_table[color].blue);
+       } else {
+           SP->_color_table[color].red = r;
+           SP->_color_table[color].green = g;
+           SP->_color_table[color].blue = b;
+       }
 
-    if (initialize_color) {
        TPUTS_TRACE("initialize_color");
-       putp(tparm(initialize_color, color, r, g, b));
+       putp(TPARM_4(initialize_color, color, r, g, b));
+       SP->_color_defs = max(color + 1, SP->_color_defs);
+       result = OK;
     }
-    returnCode(OK);
+    returnCode(result);
 }
 
-bool
+NCURSES_EXPORT(bool)
 can_change_color(void)
 {
     T((T_CALLED("can_change_color()")));
     returnCode((can_change != 0) ? TRUE : FALSE);
 }
 
-bool
+NCURSES_EXPORT(bool)
 has_colors(void)
 {
     T((T_CALLED("has_colors()")));
     returnCode((VALID_NUMERIC(max_colors) && VALID_NUMERIC(max_pairs)
-           && (((set_foreground != NULL)
-                   && (set_background != NULL))
-               || ((set_a_foreground != NULL)
-                   && (set_a_background != NULL))
-               || set_color_pair)) ? TRUE : FALSE);
+               && (((set_foreground != NULL)
+                    && (set_background != NULL))
+                   || ((set_a_foreground != NULL)
+                       && (set_a_background != NULL))
+                   || set_color_pair)) ? TRUE : FALSE);
 }
 
-int
+NCURSES_EXPORT(int)
 color_content(short color, short *r, short *g, short *b)
 {
-    T((T_CALLED("color_content(%d,%p,%p,%p)"), color, r, g, b));
-    if (color < 0 || color >= COLORS)
-       returnCode(ERR);
+    int result;
 
-    if (r)
-       *r = SP->_color_table[color].red;
-    if (g)
-       *g = SP->_color_table[color].green;
-    if (b)
-       *b = SP->_color_table[color].blue;
-    returnCode(OK);
+    T((T_CALLED("color_content(%d,%p,%p,%p)"), color, r, g, b));
+    if (color < 0 || !OkColorHi(color) || SP == 0 || !SP->_coloron) {
+       result = ERR;
+    } else {
+       NCURSES_COLOR_T c_r = SP->_color_table[color].red;
+       NCURSES_COLOR_T c_g = SP->_color_table[color].green;
+       NCURSES_COLOR_T c_b = SP->_color_table[color].blue;
+
+       if (r)
+           *r = c_r;
+       if (g)
+           *g = c_g;
+       if (b)
+           *b = c_b;
+
+       TR(TRACE_ATTRS, ("...color_content(%d,%d,%d,%d)",
+                        color, c_r, c_g, c_b));
+       result = OK;
+    }
+    returnCode(result);
 }
 
-int
+NCURSES_EXPORT(int)
 pair_content(short pair, short *f, short *b)
 {
+    int result;
+
     T((T_CALLED("pair_content(%d,%p,%p)"), pair, f, b));
 
-    if ((pair < 0) || (pair >= COLOR_PAIRS))
-       returnCode(ERR);
-    if (f)
-       *f = ((SP->_color_pairs[pair] >> C_SHIFT) & C_MASK);
-    if (b)
-       *b = (SP->_color_pairs[pair] & C_MASK);
+    if ((pair < 0) || (pair >= COLOR_PAIRS) || SP == 0 || !SP->_coloron) {
+       result = ERR;
+    } else {
+       NCURSES_COLOR_T fg = ((SP->_color_pairs[pair] >> C_SHIFT) & C_MASK);
+       NCURSES_COLOR_T bg = (SP->_color_pairs[pair] & C_MASK);
+
+#if NCURSES_EXT_FUNCS
+       if (fg == COLOR_DEFAULT)
+           fg = -1;
+       if (bg == COLOR_DEFAULT)
+           bg = -1;
+#endif
 
-    returnCode(OK);
+       if (f)
+           *f = fg;
+       if (b)
+           *b = bg;
+
+       TR(TRACE_ATTRS, ("...pair_content(%d,%d,%d)", pair, fg, bg));
+       result = OK;
+    }
+    returnCode(result);
 }
 
-void
-_nc_do_color(int old_pair, int pair, bool reverse, int (*outc) (int))
+NCURSES_EXPORT(void)
+_nc_do_color(short old_pair, short pair, bool reverse, int (*outc) (int))
 {
-    NCURSES_COLOR_T fg = C_MASK, bg = C_MASK;
+    NCURSES_COLOR_T fg = COLOR_DEFAULT;
+    NCURSES_COLOR_T bg = COLOR_DEFAULT;
     NCURSES_COLOR_T old_fg, old_bg;
 
     if (pair < 0 || pair >= COLOR_PAIRS) {
@@ -427,41 +532,46 @@ _nc_do_color(int old_pair, int pair, bool reverse, int (*outc) (int))
     } else if (pair != 0) {
        if (set_color_pair) {
            TPUTS_TRACE("set_color_pair");
-           tputs(tparm(set_color_pair, pair), 1, outc);
+           tputs(TPARM_1(set_color_pair, pair), 1, outc);
            return;
        } else if (SP != 0) {
-           pair_content(pair, &fg, &bg);
+           pair_content((short) pair, &fg, &bg);
        }
     }
 
-    if (old_pair >= 0 && SP != 0) {
-       pair_content(old_pair, &old_fg, &old_bg);
-       if ((fg == C_MASK && old_fg != C_MASK)
-           || (bg == C_MASK && old_bg != C_MASK)) {
-#ifdef NCURSES_EXT_FUNCS
+    if (old_pair >= 0
+       && SP != 0
+       && pair_content(old_pair, &old_fg, &old_bg) != ERR) {
+       if ((isDefaultColor(fg) && !isDefaultColor(old_fg))
+           || (isDefaultColor(bg) && !isDefaultColor(old_bg))) {
+#if NCURSES_EXT_FUNCS
            /*
             * A minor optimization - but extension.  If "AX" is specified in
             * the terminal description, treat it as screen's indicator of ECMA
             * SGR 39 and SGR 49, and assume the two sequences are independent.
             */
-           if (SP->_has_sgr_39_49 && old_bg == C_MASK && old_fg != C_MASK) {
+           if (SP->_has_sgr_39_49
+               && isDefaultColor(old_bg)
+               && !isDefaultColor(old_fg)) {
                tputs("\033[39m", 1, outc);
-           } else if (SP->_has_sgr_39_49 && old_fg == C_MASK && old_bg != C_MASK) {
+           } else if (SP->_has_sgr_39_49
+                      && isDefaultColor(old_fg)
+                      && !isDefaultColor(old_bg)) {
                tputs("\033[49m", 1, outc);
            } else
 #endif
-               set_original_colors();
+               reset_color_pair();
        }
     } else {
-       set_original_colors();
+       reset_color_pair();
        if (old_pair < 0)
            return;
     }
 
-#ifdef NCURSES_EXT_FUNCS
-    if (fg == C_MASK)
+#if NCURSES_EXT_FUNCS
+    if (isDefaultColor(fg))
        fg = default_fg();
-    if (bg == C_MASK)
+    if (isDefaultColor(bg))
        bg = default_bg();
 #endif
 
@@ -471,12 +581,13 @@ _nc_do_color(int old_pair, int pair, bool reverse, int (*outc) (int))
        bg = xx;
     }
 
-    T(("setting colors: pair = %d, fg = %d, bg = %d", pair, fg, bg));
+    TR(TRACE_ATTRS, ("setting colors: pair = %d, fg = %d, bg = %d", pair,
+                    fg, bg));
 
-    if (fg != C_MASK) {
+    if (!isDefaultColor(fg)) {
        set_foreground_color(fg, outc);
     }
-    if (bg != C_MASK) {
+    if (!isDefaultColor(bg)) {
        set_background_color(bg, outc);
     }
 }