]> granicus.if.org Git - neomutt/commitdiff
refactor: NORMAL_COLOR() to mutt_curses_set_color()
authorRichard Russon <rich@flatcap.org>
Fri, 27 Sep 2019 12:51:34 +0000 (13:51 +0100)
committerRichard Russon <rich@flatcap.org>
Sat, 28 Sep 2019 02:18:27 +0000 (03:18 +0100)
Encapsulate a curses function to reduce dependencies.

compose.c
curs_lib.c
index.c
main.c
menu.c
mutt_curses.h
mutt_logging.c
pager.c
progress.c
remailer.c
sidebar.c

index dd46652e357b3fb39256b1a651762a7116998a57..5323e12dfa8cfec8e74ee6a53ecc85a29ea535ee 100644 (file)
--- a/compose.c
+++ b/compose.c
@@ -373,7 +373,7 @@ static void redraw_crypt_lines(struct ComposeRedrawData *rd)
   mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
   mutt_window_mvprintw(rd->win, HDR_CRYPT, 0, "%*s", HeaderPadding[HDR_CRYPT],
                        _(Prompts[HDR_CRYPT]));
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 
   if ((WithCrypto & (APPLICATION_PGP | APPLICATION_SMIME)) == 0)
   {
@@ -402,7 +402,7 @@ static void redraw_crypt_lines(struct ComposeRedrawData *rd)
     mutt_curses_set_color(MT_COLOR_COMPOSE_SECURITY_NONE);
     addstr(_("None"));
   }
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 
   if ((e->security & (SEC_ENCRYPT | SEC_SIGN)))
   {
@@ -429,7 +429,7 @@ static void redraw_crypt_lines(struct ComposeRedrawData *rd)
   {
     mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
     printw("%*s", HeaderPadding[HDR_CRYPTINFO], _(Prompts[HDR_CRYPTINFO]));
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     printw("%s", C_PgpSignAs ? C_PgpSignAs : _("<default>"));
   }
 
@@ -438,7 +438,7 @@ static void redraw_crypt_lines(struct ComposeRedrawData *rd)
   {
     mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
     printw("%*s", HeaderPadding[HDR_CRYPTINFO], _(Prompts[HDR_CRYPTINFO]));
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     printw("%s", C_SmimeSignAs ? C_SmimeSignAs : _("<default>"));
   }
 
@@ -447,7 +447,7 @@ static void redraw_crypt_lines(struct ComposeRedrawData *rd)
   {
     mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
     mutt_window_mvprintw(rd->win, HDR_CRYPTINFO, 40, "%s", _("Encrypt with: "));
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     printw("%s", NONULL(C_SmimeEncryptWith));
   }
 
@@ -458,7 +458,7 @@ static void redraw_crypt_lines(struct ComposeRedrawData *rd)
   {
     mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
     printw("%*s", HeaderPadding[HDR_AUTOCRYPT], _(Prompts[HDR_AUTOCRYPT]));
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     if (e->security & SEC_AUTOCRYPT)
     {
       mutt_curses_set_color(MT_COLOR_COMPOSE_SECURITY_ENCRYPT);
@@ -478,7 +478,7 @@ static void redraw_crypt_lines(struct ComposeRedrawData *rd)
                              (Off, No, Discouraged, Available, Yes)
                           */
                          _("Recommendation: "));
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     printw("%s", _(AutocryptRecUiFlags[rd->autocrypt_rec]));
   }
 #endif
@@ -537,7 +537,7 @@ static void redraw_mix_line(struct ListHead *chain, struct ComposeRedrawData *rd
   mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
   mutt_window_mvprintw(rd->win, HDR_MIX, 0, "%*s", HeaderPadding[HDR_MIX],
                        _(Prompts[HDR_MIX]));
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 
   if (STAILQ_EMPTY(chain))
   {
@@ -619,7 +619,7 @@ static void draw_envelope_addr(int line, struct AddressList *al, struct ComposeR
   mutt_addrlist_write(buf, sizeof(buf), al, true);
   mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
   mutt_window_mvprintw(rd->win, line, 0, "%*s", HeaderPadding[line], _(Prompts[line]));
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   mutt_paddstr(W, buf);
 }
 
@@ -662,7 +662,7 @@ static void draw_envelope(struct ComposeRedrawData *rd)
   mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
   mutt_window_mvprintw(rd->win, HDR_SUBJECT, 0, "%*s",
                        HeaderPadding[HDR_SUBJECT], _(Prompts[HDR_SUBJECT]));
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   mutt_paddstr(W, NONULL(e->env->subject));
 
   draw_envelope_addr(HDR_REPLYTO, &e->env->reply_to, rd);
@@ -670,7 +670,7 @@ static void draw_envelope(struct ComposeRedrawData *rd)
   mutt_curses_set_color(MT_COLOR_COMPOSE_HEADER);
   mutt_window_mvprintw(rd->win, HDR_FCC, 0, "%*s", HeaderPadding[HDR_FCC],
                        _(Prompts[HDR_FCC]));
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   mutt_paddstr(W, fcc);
 
   if (WithCrypto)
@@ -684,7 +684,7 @@ static void draw_envelope(struct ComposeRedrawData *rd)
   mutt_window_mvaddstr(rd->win, HDR_ATTACH_TITLE, 0, _("-- Attachments"));
   mutt_window_clrtoeol(rd->win);
 
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 }
 
 /**
@@ -868,7 +868,7 @@ static void compose_custom_redraw(struct Menu *menu)
     mutt_window_move(menu->statuswin, 0, 0);
     mutt_curses_set_color(MT_COLOR_STATUS);
     mutt_paddstr(menu->statuswin->cols, buf);
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     menu->redraw &= ~REDRAW_STATUS;
   }
 
index 29e6a974226e1799ed8f8a5705ab74c985031ca9..47026e7750a8e2dd1790000f5863e6eab1a70cbf 100644 (file)
@@ -264,7 +264,7 @@ int mutt_get_field_full(const char *field, char *buf, size_t buflen, CompletionF
     mutt_window_clearline(MuttMessageWindow, 0);
     mutt_curses_set_color(MT_COLOR_PROMPT);
     addstr(field);
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     mutt_refresh();
     mutt_window_getxy(MuttMessageWindow, &x, NULL);
     ret = mutt_enter_string_full(buf, buflen, x, complete, multiple, files, numfiles, es);
@@ -395,7 +395,7 @@ enum QuadOption mutt_yesorno(const char *msg, enum QuadOption def)
       mutt_curses_set_color(MT_COLOR_PROMPT);
       addnstr(msg, trunc_msg_len);
       addstr(answer_string);
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
       mutt_window_clrtoeol(MuttMessageWindow);
     }
 
@@ -490,7 +490,7 @@ void mutt_show_error(void)
 
   mutt_curses_set_color(OptMsgErr ? MT_COLOR_ERROR : MT_COLOR_MESSAGE);
   mutt_window_mvaddstr(MuttMessageWindow, 0, 0, ErrorBuf);
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   mutt_window_clrtoeol(MuttMessageWindow);
 }
 
@@ -659,7 +659,7 @@ int mutt_buffer_enter_fname_full(const char *prompt, struct Buffer *fname,
   mutt_curses_set_color(MT_COLOR_PROMPT);
   mutt_window_mvaddstr(MuttMessageWindow, 0, 0, prompt);
   addstr(_(" ('?' for list): "));
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   if (!mutt_buffer_is_empty(fname))
     addstr(mutt_b2s(fname));
   mutt_window_clrtoeol(MuttMessageWindow);
@@ -911,7 +911,7 @@ int mutt_multi_choice(const char *prompt, const char *letters)
 
       mutt_curses_set_color(MT_COLOR_PROMPT);
       addstr(prompt);
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
 
       addch(' ');
       mutt_window_clrtoeol(MuttMessageWindow);
diff --git a/index.c b/index.c
index 6d32ca29d213d7ccc7c2628a1372a2389cd01831..d989d2d8624781fdb4fba88bd854b796e86e1321 100644 (file)
--- a/index.c
+++ b/index.c
@@ -1011,7 +1011,7 @@ static void index_custom_redraw(struct Menu *menu)
     mutt_window_move(menu->statuswin, 0, 0);
     mutt_curses_set_color(MT_COLOR_STATUS);
     mutt_draw_statusline(menu->statuswin->cols, buf, sizeof(buf));
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     menu->redraw &= ~REDRAW_STATUS;
     if (C_TsEnabled && TsSupported)
     {
diff --git a/main.c b/main.c
index 8c738917ebf22fc6fde017190e113dce496cccfa..1b7e8f56c3c7daf89735adb38e348f4f94402a27 100644 (file)
--- a/main.c
+++ b/main.c
@@ -790,7 +790,7 @@ int main(int argc, char *argv[], char *envp[])
 
   if (!OptNoCurses)
   {
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     clear();
     MuttLogger = log_disp_curses;
     log_queue_flush(log_disp_curses);
diff --git a/menu.c b/menu.c
index a5caa6753697b8fd6579707f4844febb4ab04e43..e210fe7f3a53fe195f6843bab2aef2fce3833136 100644 (file)
--- a/menu.c
+++ b/menu.c
@@ -360,7 +360,7 @@ static void menu_pad_string(struct Menu *menu, char *buf, size_t buflen)
  */
 void menu_redraw_full(struct Menu *menu)
 {
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   /* clear() doesn't optimize screen redraws */
   move(0, 0);
   clrtobot();
@@ -370,7 +370,7 @@ void menu_redraw_full(struct Menu *menu)
     mutt_curses_set_color(MT_COLOR_STATUS);
     mutt_window_move(MuttHelpWindow, 0, 0);
     mutt_paddstr(MuttHelpWindow->cols, menu->help);
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
   }
   menu->offset = 0;
   menu->pagelen = menu->indexwin->rows;
@@ -395,7 +395,7 @@ void menu_redraw_status(struct Menu *menu)
   mutt_curses_set_color(MT_COLOR_STATUS);
   mutt_window_move(menu->statuswin, 0, 0);
   mutt_paddstr(menu->statuswin->cols, buf);
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   menu->redraw &= ~REDRAW_STATUS;
 }
 
@@ -453,11 +453,11 @@ void menu_redraw_index(struct Menu *menu)
     }
     else
     {
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
       mutt_window_clearline(menu->indexwin, i - menu->top + menu->offset);
     }
   }
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
   menu->redraw = 0;
 }
 
@@ -516,7 +516,7 @@ void menu_redraw_motion(struct Menu *menu)
     print_enriched_string(menu->current, cur_color, (unsigned char *) buf, false);
   }
   menu->redraw &= REDRAW_STATUS;
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 }
 
 /**
@@ -544,7 +544,7 @@ void menu_redraw_current(struct Menu *menu)
   else
     print_enriched_string(menu->current, attr, (unsigned char *) buf, false);
   menu->redraw &= REDRAW_STATUS;
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 }
 
 /**
index 7ba196586f34db136f0ed0a54d7f83b2f5d7b1d0..6c76e01f9e24d3459685fffaa5c07d152ae566b3 100644 (file)
@@ -111,11 +111,4 @@ void mutt_resize_screen(void);
 #define ATTR_SET attrset
 #endif
 
-/* reset the color to the normal terminal color as defined by 'color normal ...' */
-#ifdef HAVE_BKGDSET
-#define NORMAL_COLOR bkgdset(ColorDefs[MT_COLOR_NORMAL] | ' ')
-#else
-#define NORMAL_COLOR attrset(ColorDefs[MT_COLOR_NORMAL])
-#endif
-
 #endif /* MUTT_MUTT_CURSES_H */
index ba9b88be890393f8f5e817306e464db4d9a9534c..b75396b2520c0aab3d0444ad178dd81d284c2ccf 100644 (file)
@@ -174,7 +174,7 @@ int log_disp_curses(time_t stamp, const char *file, int line,
       BEEP();
     mutt_curses_set_color((level == LL_ERROR) ? MT_COLOR_ERROR : MT_COLOR_MESSAGE);
     mutt_window_mvaddstr(MuttMessageWindow, 0, 0, ErrorBuf);
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     mutt_window_clrtoeol(MuttMessageWindow);
     mutt_refresh();
   }
diff --git a/pager.c b/pager.c
index c9c746e4b65b047d0d10e0753f29326fd2df93b6..84743ce7d93ded7a9eb15460778423d8371f6420 100644 (file)
--- a/pager.c
+++ b/pager.c
@@ -1855,7 +1855,7 @@ static int display_line(FILE *fp, LOFF_T *last_pos, struct Line **line_info,
 #ifndef USE_SLANG_CURSES
   if (col == 0)
   {
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     addch(' ');
   }
 #endif
@@ -1885,7 +1885,7 @@ static int display_line(FILE *fp, LOFF_T *last_pos, struct Line **line_info,
    * clrtoeol, otherwise the color for this line will not be
    * filled to the right margin.  */
   if (flags & MUTT_SHOWCOLOR)
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
 
   /* build a return code */
   if (!(flags & MUTT_SHOW))
@@ -1941,7 +1941,7 @@ static void pager_custom_redraw(struct Menu *pager_menu)
 
   if (pager_menu->redraw & REDRAW_FULL)
   {
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     /* clear() doesn't optimize screen redraws */
     move(0, 0);
     clrtobot();
@@ -1994,7 +1994,7 @@ static void pager_custom_redraw(struct Menu *pager_menu)
       mutt_curses_set_color(MT_COLOR_STATUS);
       mutt_window_move(MuttHelpWindow, 0, 0);
       mutt_paddstr(MuttHelpWindow->cols, rd->helpstr);
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
     }
 
     if (Resize)
@@ -2037,7 +2037,7 @@ static void pager_custom_redraw(struct Menu *pager_menu)
         rd->menu->statuswin = rd->index_status_window;
       }
 
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
       rd->menu->pagelen = rd->index_window->rows;
 
       /* some fudge to work out whereabouts the indicator should go */
@@ -2143,7 +2143,7 @@ static void pager_custom_redraw(struct Menu *pager_menu)
       rd->lines++;
       mutt_window_move(rd->pager_window, rd->lines, 0);
     }
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
 
     /* We are going to update the pager status bar, so it isn't
      * necessary to reset to normal color now. */
@@ -2194,7 +2194,7 @@ static void pager_custom_redraw(struct Menu *pager_menu)
       snprintf(bn, sizeof(bn), "%s (%s)", rd->banner, pager_progress_str);
       mutt_draw_statusline(rd->pager_status_window->cols, bn, sizeof(bn));
     }
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
     if (C_TsEnabled && TsSupported && rd->menu)
     {
       menu_status_line(buf, sizeof(buf), rd->menu, NONULL(C_TsStatusFormat));
@@ -2217,7 +2217,7 @@ static void pager_custom_redraw(struct Menu *pager_menu)
     mutt_window_move(rd->index_status_window, 0, 0);
     mutt_curses_set_color(MT_COLOR_STATUS);
     mutt_draw_statusline(rd->index_status_window->cols, buf, sizeof(buf));
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
   }
 
   pager_menu->redraw = REDRAW_NO_FLAGS;
index d86fa8520ec93cbb52fd0cfc064d0f26f925d73c..281a087042f3c289a1e56bc7bfd522135eff5823 100644 (file)
@@ -88,7 +88,7 @@ static void message_bar(int percent, const char *fmt, ...)
       {
         addch(' ');
       }
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
     }
     else
     {
@@ -100,7 +100,7 @@ static void message_bar(int percent, const char *fmt, ...)
       mutt_curses_set_color(MT_COLOR_PROGRESS);
       addstr(buf2);
       buf2[off] = ch;
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
       addstr(&buf2[off]);
     }
   }
index 0cb794d6b2ca881acab9fe956d80f4911e7261dd..8215cdb53f61041b84e9817b2c20846aeedc4540 100644 (file)
@@ -333,11 +333,11 @@ static void mix_redraw_ce(struct Remailer **type2_list, struct Coord *coords,
     if (selected)
       mutt_curses_set_color(MT_COLOR_INDICATOR);
     else
-      NORMAL_COLOR;
+      mutt_curses_set_color(MT_COLOR_NORMAL);
 
     mutt_window_mvaddstr(MuttIndexWindow, coords[i].r, coords[i].c,
                          type2_list[chain->ch[i]]->shortname);
-    NORMAL_COLOR;
+    mutt_curses_set_color(MT_COLOR_NORMAL);
 
     if (i + 1 < chain->cl)
       addstr(", ");
@@ -374,7 +374,7 @@ static void mix_redraw_head(struct MixChain *chain)
   mutt_window_mvprintw(MuttIndexWindow, MIX_VOFFSET - 1, 0,
                        "-- Remailer chain [Length: %d]", chain ? chain->cl : 0);
   mutt_window_clrtoeol(MuttIndexWindow);
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 }
 
 /**
index 8b2ae9e9cc8582c06b49e307b6ba73241374d24d..7a12ff69750f003432fcd32d8b28a9c24261a644 100644 (file)
--- a/sidebar.c
+++ b/sidebar.c
@@ -784,7 +784,7 @@ static int draw_divider(int num_rows, int num_cols)
 static void fill_empty_space(int first_row, int num_rows, int div_width, int num_cols)
 {
   /* Fill the remaining rows with blank space */
-  NORMAL_COLOR;
+  mutt_curses_set_color(MT_COLOR_NORMAL);
 
   if (!C_SidebarOnRight)
     div_width = 0;
@@ -856,7 +856,7 @@ static void draw_sidebar(int num_rows, int num_cols, int div_width)
       if (ColorDefs[MT_COLOR_ORDINARY] != 0)
         mutt_curses_set_color(MT_COLOR_ORDINARY);
       else
-        NORMAL_COLOR;
+        mutt_curses_set_color(MT_COLOR_NORMAL);
     }
 
     int col = 0;