]> ncurses.scripts.mit.edu Git - ncurses.git/blobdiff - ncurses/base/lib_color.c
ncurses 6.0 - patch 20170401
[ncurses.git] / ncurses / base / lib_color.c
index 003929871f99fe8962f5313f9ec42ae2d4f67e3f..72a2a4857559dee5e11878f850dea736d3937540 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
- * Copyright (c) 1998-2011,2012 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2016,2017 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            *
  * Handles color emulation of SYS V curses
  */
 
+#define NEW_PAIR_INTERNAL 1
+
 #include <curses.priv.h>
+#include <new_pair.h>
 #include <tic.h>
 
 #ifndef CUR
 #define CUR SP_TERMTYPE
 #endif
 
-MODULE_ID("$Id: lib_color.c,v 1.106 2012/12/15 19:14:34 tom Exp $")
+MODULE_ID("$Id: lib_color.c,v 1.116 2017/03/25 21:10:54 tom Exp $")
 
 #ifdef USE_TERM_DRIVER
 #define CanChange      InfoOf(SP_PARM).canchange
@@ -137,13 +140,6 @@ NCURSES_EXPORT_VAR(const color_t*) _nc_hls_palette = hls_palette;
 
 /* *INDENT-ON* */
 
-/*
- * Ensure that we use color pairs only when colors have been started, and also
- * that the index is within the limits of the table which we allocated.
- */
-#define ValidPair(pair) \
-    ((SP_PARM != 0) && (pair >= 0) && (pair < SP_PARM->_pair_limit) && SP_PARM->_coloron)
-
 #if NCURSES_EXT_FUNCS
 /*
  * These are called from _nc_do_color(), which in turn is called from
@@ -188,7 +184,7 @@ static void
 set_background_color(NCURSES_SP_DCLx int bg, NCURSES_SP_OUTC outc)
 {
 #ifdef USE_TERM_DRIVER
-    CallDriver_3(SP_PARM, color, FALSE, bg, outc);
+    CallDriver_3(SP_PARM, td_color, FALSE, bg, outc);
 #else
     if (set_a_background) {
        TPUTS_TRACE("set_a_background");
@@ -208,7 +204,7 @@ static void
 set_foreground_color(NCURSES_SP_DCLx int fg, NCURSES_SP_OUTC outc)
 {
 #ifdef USE_TERM_DRIVER
-    CallDriver_3(SP_PARM, color, TRUE, fg, outc);
+    CallDriver_3(SP_PARM, td_color, TRUE, fg, outc);
 #else
     if (set_a_foreground) {
        TPUTS_TRACE("set_a_foreground");
@@ -258,14 +254,13 @@ static bool
 reset_color_pair(NCURSES_SP_DCL0)
 {
 #ifdef USE_TERM_DRIVER
-    return CallDriver(SP_PARM, rescol);
+    return CallDriver(SP_PARM, td_rescol);
 #else
     bool result = FALSE;
 
     (void) SP_PARM;
     if (orig_pair != 0) {
-       TPUTS_TRACE("orig_pair");
-       putp(orig_pair);
+       (void) NCURSES_PUTP2("orig_pair", orig_pair);
        result = TRUE;
     }
     return result;
@@ -289,11 +284,10 @@ NCURSES_SP_NAME(_nc_reset_colors) (NCURSES_SP_DCL0)
        result = TRUE;
 
 #ifdef USE_TERM_DRIVER
-    result = CallDriver(SP_PARM, rescolors);
+    result = CallDriver(SP_PARM, td_rescolors);
 #else
     if (orig_colors != 0) {
-       TPUTS_TRACE("orig_colors");
-       putp(orig_colors);
+       NCURSES_PUTP2("orig_colors", orig_colors);
        result = TRUE;
     }
 #endif
@@ -348,6 +342,7 @@ NCURSES_SP_NAME(start_color) (NCURSES_SP_DCL0)
             * allow for default-color as a component of a color-pair.
             */
            SP_PARM->_pair_limit += (1 + (2 * maxcolors));
+           SP_PARM->_pair_limit = limit_PAIRS(SP_PARM->_pair_limit);
 #endif
            SP_PARM->_pair_count = maxpairs;
            SP_PARM->_color_count = maxcolors;
@@ -360,8 +355,9 @@ NCURSES_SP_NAME(start_color) (NCURSES_SP_DCL0)
            if (SP_PARM->_color_pairs != 0) {
                SP_PARM->_color_table = TYPE_CALLOC(color_t, maxcolors);
                if (SP_PARM->_color_table != 0) {
-                   SP_PARM->_color_pairs[0] = PAIR_OF(default_fg(NCURSES_SP_ARG),
-                                                      default_bg(NCURSES_SP_ARG));
+                   MakeColorPair(SP_PARM->_color_pairs[0],
+                                 default_fg(NCURSES_SP_ARG),
+                                 default_bg(NCURSES_SP_ARG));
                    init_color_table(NCURSES_SP_ARG);
 
                    T(("started color: COLORS = %d, COLOR_PAIRS = %d",
@@ -390,7 +386,7 @@ start_color(void)
 
 /* This function was originally written by Daniel Weaver <danw@znyx.com> */
 static void
-rgb2hls(int r, int g, int b, short *h, short *l, short *s)
+rgb2hls(int r, int g, int b, int *h, int *l, int *s)
 /* convert RGB to HLS system */
 {
     int min, max, t;
@@ -401,7 +397,7 @@ rgb2hls(int r, int g, int b, short *h, short *l, short *s)
        max = b;
 
     /* calculate lightness */
-    *l = (short) ((min + max) / 20);
+    *l = ((min + max) / 20);
 
     if (min == max) {          /* black, white and all shades of gray */
        *h = 0;
@@ -411,19 +407,44 @@ rgb2hls(int r, int g, int b, short *h, short *l, short *s)
 
     /* calculate saturation */
     if (*l < 50)
-       *s = (short) (((max - min) * 100) / (max + min));
+       *s = (((max - min) * 100) / (max + min));
     else
-       *s = (short) (((max - min) * 100) / (2000 - max - min));
+       *s = (((max - min) * 100) / (2000 - max - min));
 
     /* calculate hue */
     if (r == max)
-       t = (short) (120 + ((g - b) * 60) / (max - min));
+       t = (120 + ((g - b) * 60) / (max - min));
     else if (g == max)
-       t = (short) (240 + ((b - r) * 60) / (max - min));
+       t = (240 + ((b - r) * 60) / (max - min));
     else
-       t = (short) (360 + ((r - g) * 60) / (max - min));
+       t = (360 + ((r - g) * 60) / (max - min));
+
+    *h = (t % 360);
+}
 
-    *h = (short) (t % 360);
+/*
+ * Change all cells which use(d) a given color pair to force a repaint.
+ */
+NCURSES_EXPORT(void)
+_nc_change_pair(SCREEN *sp, int pair)
+{
+    int y, x;
+
+    for (y = 0; y <= CurScreen(sp)->_maxy; y++) {
+       struct ldat *ptr = &(CurScreen(sp)->_line[y]);
+       bool changed = FALSE;
+       for (x = 0; x <= CurScreen(sp)->_maxx; x++) {
+           if (GetPair(ptr->text[x]) == pair) {
+               /* Set the old cell to zero to ensure it will be
+                  updated on the next doupdate() */
+               SetChar(ptr->text[x], 0, 0);
+               CHANGED_CELL(ptr, x);
+               changed = TRUE;
+           }
+       }
+       if (changed)
+           NCURSES_SP_NAME(_nc_make_oldhash) (NCURSES_SP_ARGx y);
+    }
 }
 
 /*
@@ -431,25 +452,26 @@ rgb2hls(int r, int g, int b, short *h, short *l, short *s)
  * values.
  */
 NCURSES_EXPORT(int)
-NCURSES_SP_NAME(init_pair) (NCURSES_SP_DCLx short pair, short f, short b)
+_nc_init_pair(SCREEN *sp, int pair, int f, int b)
 {
-    colorpair_t result;
+    static colorpair_t null_pair;
+    colorpair_t result = null_pair;
     colorpair_t previous;
     int maxcolors;
 
-    T((T_CALLED("init_pair(%p,%d,%d,%d)"), (void *) SP_PARM, pair, f, b));
+    T((T_CALLED("init_pair(%p,%d,%d,%d)"), (void *) sp, pair, f, b));
 
-    if (!ValidPair(pair))
+    if (!ValidPair(sp, pair))
        returnCode(ERR);
 
     maxcolors = MaxColors;
 
-    previous = SP_PARM->_color_pairs[pair];
+    previous = sp->_color_pairs[pair];
 #if NCURSES_EXT_FUNCS
-    if (SP_PARM->_default_color || SP_PARM->_assumed_color) {
+    if (sp->_default_color || sp->_assumed_color) {
        bool isDefault = FALSE;
        bool wasDefault = FALSE;
-       int default_pairs = SP_PARM->_default_pairs;
+       int default_pairs = sp->_default_pairs;
 
        /*
         * Map caller's color number, e.g., -1, 0, 1, .., 7, etc., into
@@ -474,8 +496,8 @@ NCURSES_SP_NAME(init_pair) (NCURSES_SP_DCLx short pair, short f, short b)
         * Check if the table entry that we are going to init/update used
         * default colors.
         */
-       if ((FORE_OF(previous) == COLOR_DEFAULT)
-           || (BACK_OF(previous) == COLOR_DEFAULT))
+       if (isDefaultColor(FORE_OF(previous))
+           || isDefaultColor(BACK_OF(previous)))
            wasDefault = TRUE;
 
        /*
@@ -496,10 +518,10 @@ NCURSES_SP_NAME(init_pair) (NCURSES_SP_DCLx short pair, short f, short b)
         * that does not will decrement the count - and possibly interfere
         * with sequentially adding new pairs.
         */
-       if (pair > (SP_PARM->_pair_count + default_pairs)) {
+       if (pair > (sp->_pair_count + default_pairs)) {
            returnCode(ERR);
        }
-       SP_PARM->_default_pairs = default_pairs;
+       sp->_default_pairs = default_pairs;
     } else
 #endif
     {
@@ -515,58 +537,59 @@ NCURSES_SP_NAME(init_pair) (NCURSES_SP_DCLx short pair, short f, short b)
      * initialized before a screen update is performed replacing original
      * pair colors with the new ones).
      */
-    result = PAIR_OF(f, b);
-    if (previous != 0
-       && previous != result) {
-       int y, x;
-
-       for (y = 0; y <= CurScreen(SP_PARM)->_maxy; y++) {
-           struct ldat *ptr = &(CurScreen(SP_PARM)->_line[y]);
-           bool changed = FALSE;
-           for (x = 0; x <= CurScreen(SP_PARM)->_maxx; x++) {
-               if (GetPair(ptr->text[x]) == pair) {
-                   /* Set the old cell to zero to ensure it will be
-                      updated on the next doupdate() */
-                   SetChar(ptr->text[x], 0, 0);
-                   CHANGED_CELL(ptr, x);
-                   changed = TRUE;
-               }
-           }
-           if (changed)
-               NCURSES_SP_NAME(_nc_make_oldhash) (NCURSES_SP_ARGx y);
-       }
+    MakeColorPair(result, f, b);
+    if (FORE_OF(previous) != 0
+       && BACK_OF(previous) != 0
+       && !isSamePair(previous, result)) {
+       _nc_change_pair(sp, pair);
     }
 
-    SP_PARM->_color_pairs[pair] = result;
-    if (GET_SCREEN_PAIR(SP_PARM) == pair)
-       SET_SCREEN_PAIR(SP_PARM, (chtype) (~0));        /* force attribute update */
+    _nc_reset_color_pair(sp, pair, &result);
+    sp->_color_pairs[pair] = result;
+    _nc_set_color_pair(sp, pair, cpINIT);
+
+    if (GET_SCREEN_PAIR(sp) == pair)
+       SET_SCREEN_PAIR(sp, (int) (~0));        /* force attribute update */
 
 #ifdef USE_TERM_DRIVER
-    CallDriver_3(SP_PARM, initpair, pair, f, b);
+    CallDriver_3(sp, td_initpair, pair, f, b);
 #else
     if (initialize_pair && InPalette(f) && InPalette(b)) {
        const color_t *tp = DefaultPalette;
 
        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));
+           (int) pair,
+           (int) tp[f].red, (int) tp[f].green, (int) tp[f].blue,
+           (int) tp[b].red, (int) tp[b].green, (int) tp[b].blue));
+
+       NCURSES_PUTP2("initialize_pair",
+                     TPARM_7(initialize_pair,
+                             pair,
+                             (int) tp[f].red,
+                             (int) tp[f].green,
+                             (int) tp[f].blue,
+                             (int) tp[b].red,
+                             (int) tp[b].green,
+                             (int) tp[b].blue));
     }
 #endif
 
     returnCode(OK);
 }
 
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(init_pair) (NCURSES_SP_DCLx
+                           NCURSES_PAIRS_T pair,
+                           NCURSES_COLOR_T f,
+                           NCURSES_COLOR_T b)
+{
+    return _nc_init_pair(SP_PARM, pair, f, b);
+}
+
 #if NCURSES_SP_FUNCS
 NCURSES_EXPORT(int)
-init_pair(short pair, short f, short b)
+init_pair(NCURSES_COLOR_T pair, NCURSES_COLOR_T f, NCURSES_COLOR_T b)
 {
     return NCURSES_SP_NAME(init_pair) (CURRENT_SCREEN, pair, f, b);
 }
@@ -575,59 +598,71 @@ init_pair(short pair, short f, short b)
 #define okRGB(n) ((n) >= 0 && (n) <= 1000)
 
 NCURSES_EXPORT(int)
-NCURSES_SP_NAME(init_color) (NCURSES_SP_DCLx
-                            short color, short r, short g, short b)
+_nc_init_color(SCREEN *sp, int color, int r, int g, int b)
 {
     int result = ERR;
     int maxcolors;
 
     T((T_CALLED("init_color(%p,%d,%d,%d,%d)"),
-       (void *) SP_PARM,
+       (void *) sp,
        color,
        r, g, b));
 
-    if (SP_PARM == 0)
+    if (sp == 0)
        returnCode(result);
 
     maxcolors = MaxColors;
 
     if (InitColor
-       && SP_PARM->_coloron
+       && sp->_coloron
        && (color >= 0 && OkColorHi(color))
        && (okRGB(r) && okRGB(g) && okRGB(b))) {
 
-       SP_PARM->_color_table[color].init = 1;
-       SP_PARM->_color_table[color].r = r;
-       SP_PARM->_color_table[color].g = g;
-       SP_PARM->_color_table[color].b = 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 (UseHlsPalette) {
            rgb2hls(r, g, b,
-                   &SP_PARM->_color_table[color].red,
-                   &SP_PARM->_color_table[color].green,
-                   &SP_PARM->_color_table[color].blue);
+                   &sp->_color_table[color].red,
+                   &sp->_color_table[color].green,
+                   &sp->_color_table[color].blue);
        } else {
-           SP_PARM->_color_table[color].red = r;
-           SP_PARM->_color_table[color].green = g;
-           SP_PARM->_color_table[color].blue = b;
+           sp->_color_table[color].red = r;
+           sp->_color_table[color].green = g;
+           sp->_color_table[color].blue = b;
        }
 
 #ifdef USE_TERM_DRIVER
-       CallDriver_4(SP_PARM, initcolor, color, r, g, b);
+       CallDriver_4(sp, td_initcolor, color, r, g, b);
 #else
-       TPUTS_TRACE("initialize_color");
-       putp(TPARM_4(initialize_color, color, r, g, b));
+       NCURSES_PUTP2("initialize_color",
+                     TPARM_4(initialize_color, color, r, g, b));
 #endif
-       SP_PARM->_color_defs = max(color + 1, SP_PARM->_color_defs);
+       sp->_color_defs = max(color + 1, sp->_color_defs);
 
        result = OK;
     }
     returnCode(result);
 }
 
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(init_color) (NCURSES_SP_DCLx
+                            NCURSES_COLOR_T color,
+                            NCURSES_COLOR_T r,
+                            NCURSES_COLOR_T g,
+                            NCURSES_COLOR_T b)
+{
+    return _nc_init_color(SP_PARM, color, r, g, b);
+}
+
 #if NCURSES_SP_FUNCS
 NCURSES_EXPORT(int)
-init_color(short color, short r, short g, short b)
+init_color(NCURSES_COLOR_T color,
+          NCURSES_COLOR_T r,
+          NCURSES_COLOR_T g,
+          NCURSES_COLOR_T b)
 {
     return NCURSES_SP_NAME(init_color) (CURRENT_SCREEN, color, r, g, b);
 }
@@ -685,31 +720,30 @@ has_colors(void)
 }
 #endif
 
-NCURSES_EXPORT(int)
-NCURSES_SP_NAME(color_content) (NCURSES_SP_DCLx
-                               short color, short *r, short *g, short *b)
+static int
+_nc_color_content(SCREEN *sp, int color, int *r, int *g, int *b)
 {
     int result = ERR;
     int maxcolors;
 
     T((T_CALLED("color_content(%p,%d,%p,%p,%p)"),
-       (void *) SP_PARM,
+       (void *) sp,
        color,
        (void *) r,
        (void *) g,
        (void *) b));
 
-    if (SP_PARM == 0)
+    if (sp == 0)
        returnCode(result);
 
     maxcolors = MaxColors;
 
-    if (color < 0 || !OkColorHi(color) || !SP_PARM->_coloron) {
+    if (color < 0 || !OkColorHi(color) || !sp->_coloron) {
        result = ERR;
     } else {
-       NCURSES_COLOR_T c_r = SP_PARM->_color_table[color].red;
-       NCURSES_COLOR_T c_g = SP_PARM->_color_table[color].green;
-       NCURSES_COLOR_T c_b = SP_PARM->_color_table[color].blue;
+       int c_r = sp->_color_table[color].red;
+       int c_g = sp->_color_table[color].green;
+       int c_b = sp->_color_table[color].blue;
 
        if (r)
            *r = c_r;
@@ -725,36 +759,55 @@ NCURSES_SP_NAME(color_content) (NCURSES_SP_DCLx
     returnCode(result);
 }
 
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(color_content) (NCURSES_SP_DCLx
+                               NCURSES_COLOR_T color,
+                               NCURSES_COLOR_T *r,
+                               NCURSES_COLOR_T *g,
+                               NCURSES_COLOR_T *b)
+{
+    int my_r, my_g, my_b;
+    int rc = _nc_color_content(SP_PARM, color, &my_r, &my_g, &my_b);
+    if (rc == OK) {
+       *r = limit_COLOR(my_r);
+       *g = limit_COLOR(my_g);
+       *b = limit_COLOR(my_b);
+    }
+    return rc;
+}
+
 #if NCURSES_SP_FUNCS
 NCURSES_EXPORT(int)
-color_content(short color, short *r, short *g, short *b)
+color_content(NCURSES_COLOR_T color,
+             NCURSES_COLOR_T *r,
+             NCURSES_COLOR_T *g,
+             NCURSES_COLOR_T *b)
 {
     return NCURSES_SP_NAME(color_content) (CURRENT_SCREEN, color, r, g, b);
 }
 #endif
 
 NCURSES_EXPORT(int)
-NCURSES_SP_NAME(pair_content) (NCURSES_SP_DCLx
-                              short pair, short *f, short *b)
+_nc_pair_content(SCREEN *sp, int pair, int *f, int *b)
 {
     int result;
 
     T((T_CALLED("pair_content(%p,%d,%p,%p)"),
-       (void *) SP_PARM,
-       pair,
+       (void *) sp,
+       (int) pair,
        (void *) f,
        (void *) b));
 
-    if (!ValidPair(pair)) {
+    if (!ValidPair(sp, pair)) {
        result = ERR;
     } else {
-       NCURSES_COLOR_T fg = (NCURSES_COLOR_T) FORE_OF(SP_PARM->_color_pairs[pair]);
-       NCURSES_COLOR_T bg = (NCURSES_COLOR_T) BACK_OF(SP_PARM->_color_pairs[pair]);
+       int fg = FORE_OF(sp->_color_pairs[pair]);
+       int bg = BACK_OF(sp->_color_pairs[pair]);
 
 #if NCURSES_EXT_FUNCS
-       if (fg == COLOR_DEFAULT)
+       if (isDefaultColor(fg))
            fg = -1;
-       if (bg == COLOR_DEFAULT)
+       if (isDefaultColor(bg))
            bg = -1;
 #endif
 
@@ -764,17 +817,32 @@ NCURSES_SP_NAME(pair_content) (NCURSES_SP_DCLx
            *b = bg;
 
        TR(TRACE_ATTRS, ("...pair_content(%p,%d,%d,%d)",
-                        (void *) SP_PARM,
-                        pair,
-                        fg, bg));
+                        (void *) sp,
+                        (int) pair,
+                        (int) fg, (int) bg));
        result = OK;
     }
     returnCode(result);
 }
 
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(pair_content) (NCURSES_SP_DCLx
+                              NCURSES_PAIRS_T pair,
+                              NCURSES_COLOR_T *f,
+                              NCURSES_COLOR_T *b)
+{
+    int my_f, my_b;
+    int rc = _nc_pair_content(SP_PARM, pair, &my_f, &my_b);
+    if (rc == OK) {
+       *f = limit_COLOR(my_f);
+       *b = limit_COLOR(my_b);
+    }
+    return rc;
+}
+
 #if NCURSES_SP_FUNCS
 NCURSES_EXPORT(int)
-pair_content(short pair, short *f, short *b)
+pair_content(NCURSES_COLOR_T pair, NCURSES_COLOR_T *f, NCURSES_COLOR_T *b)
 {
     return NCURSES_SP_NAME(pair_content) (CURRENT_SCREEN, pair, f, b);
 }
@@ -788,13 +856,14 @@ NCURSES_SP_NAME(_nc_do_color) (NCURSES_SP_DCLx
                               NCURSES_SP_OUTC outc)
 {
 #ifdef USE_TERM_DRIVER
-    CallDriver_4(SP_PARM, docolor, old_pair, pair, reverse, outc);
+    CallDriver_4(SP_PARM, td_docolor, old_pair, pair, reverse, outc);
 #else
-    NCURSES_COLOR_T fg = COLOR_DEFAULT;
-    NCURSES_COLOR_T bg = COLOR_DEFAULT;
-    NCURSES_COLOR_T old_fg, old_bg;
+    int fg = COLOR_DEFAULT;
+    int bg = COLOR_DEFAULT;
+    int old_fg = -1;
+    int old_bg = -1;
 
-    if (!ValidPair(pair)) {
+    if (!ValidPair(SP_PARM, pair)) {
        return;
     } else if (pair != 0) {
        if (set_color_pair) {
@@ -804,14 +873,14 @@ NCURSES_SP_NAME(_nc_do_color) (NCURSES_SP_DCLx
                                    1, outc);
            return;
        } else if (SP_PARM != 0) {
-           if (pair_content((short) pair, &fg, &bg) == ERR)
+           if (_nc_pair_content(SP_PARM, pair, &fg, &bg) == ERR)
                return;
        }
     }
 
     if (old_pair >= 0
        && SP_PARM != 0
-       && pair_content((short) old_pair, &old_fg, &old_bg) != ERR) {
+       && _nc_pair_content(SP_PARM, old_pair, &old_fg, &old_bg) != ERR) {
        if ((isDefaultColor(fg) && !isDefaultColor(old_fg))
            || (isDefaultColor(bg) && !isDefaultColor(old_bg))) {
 #if NCURSES_EXT_FUNCS
@@ -834,19 +903,19 @@ NCURSES_SP_NAME(_nc_do_color) (NCURSES_SP_DCLx
        }
     } else {
        reset_color_pair(NCURSES_SP_ARG);
-       if (old_pair < 0)
+       if (old_pair < 0 && pair <= 0)
            return;
     }
 
 #if NCURSES_EXT_FUNCS
     if (isDefaultColor(fg))
-       fg = (short) default_fg(NCURSES_SP_ARG);
+       fg = default_fg(NCURSES_SP_ARG);
     if (isDefaultColor(bg))
-       bg = (short) default_bg(NCURSES_SP_ARG);
+       bg = default_bg(NCURSES_SP_ARG);
 #endif
 
     if (reverse) {
-       NCURSES_COLOR_T xx = fg;
+       int xx = fg;
        fg = bg;
        bg = xx;
     }
@@ -875,3 +944,65 @@ _nc_do_color(int old_pair, int pair, int reverse, NCURSES_OUTC outc)
                                   _nc_outc_wrapper);
 }
 #endif
+
+#if USE_EXTENDED_COLORS
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(init_extended_pair) (NCURSES_SP_DCLx int pair, int f, int b)
+{
+    return _nc_init_pair(SP_PARM, pair, f, b);
+}
+
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(init_extended_color) (NCURSES_SP_DCLx
+                                     int color,
+                                     int r, int g, int b)
+{
+    return _nc_init_color(SP_PARM, color, r, g, b);
+}
+
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(extended_color_content) (NCURSES_SP_DCLx
+                                        int color,
+                                        int *r, int *g, int *b)
+{
+    return _nc_color_content(SP_PARM, color, r, g, b);
+}
+
+NCURSES_EXPORT(int)
+NCURSES_SP_NAME(extended_pair_content) (NCURSES_SP_DCLx
+                                       int pair,
+                                       int *f, int *b)
+{
+    return _nc_pair_content(SP_PARM, pair, f, b);
+}
+
+#if NCURSES_SP_FUNCS
+NCURSES_EXPORT(int)
+init_extended_pair(int pair, int f, int b)
+{
+    return NCURSES_SP_NAME(init_extended_pair) (CURRENT_SCREEN, pair, f, b);
+}
+
+NCURSES_EXPORT(int)
+init_extended_color(int color, int r, int g, int b)
+{
+    return NCURSES_SP_NAME(init_extended_color) (CURRENT_SCREEN,
+                                                color,
+                                                r, g, b);
+}
+
+NCURSES_EXPORT(int)
+extended_color_content(int color, int *r, int *g, int *b)
+{
+    return NCURSES_SP_NAME(extended_color_content) (CURRENT_SCREEN,
+                                                   color,
+                                                   r, g, b);
+}
+
+NCURSES_EXPORT(int)
+extended_pair_content(int pair, int *f, int *b)
+{
+    return NCURSES_SP_NAME(extended_pair_content) (CURRENT_SCREEN, pair, f, b);
+}
+#endif
+#endif