colors: rename to "style"
authorMax Kellermann <max@musicpd.org>
Wed, 12 Sep 2018 12:54:28 +0000 (14:54 +0200)
committerMax Kellermann <max@musicpd.org>
Wed, 12 Sep 2018 14:04:24 +0000 (16:04 +0200)
15 files changed:
meson.build
src/FileListPage.cxx
src/ListWindow.cxx
src/ListWindow.hxx
src/ProgressBar.cxx
src/StatusBar.cxx
src/Styles.cxx [moved from src/colors.cxx with 90% similarity]
src/Styles.hxx [moved from src/colors.hxx with 87% similarity]
src/TabBar.cxx
src/TitleBar.cxx
src/conf.cxx
src/ncu.cxx
src/paint.hxx
src/screen_init.cxx
src/screen_utils.cxx

index 032ade9..7436e62 100644 (file)
@@ -358,7 +358,7 @@ ncmpc = executable('ncmpc',
   'src/song_paint.cxx',
   'src/BasicColors.cxx',
   'src/CustomColors.cxx',
-  'src/colors.cxx',
+  'src/Styles.cxx',
   'src/charset.cxx',
   'src/wreadln.cxx',
   'src/Completion.cxx',
index 15f42bc..b34dae6 100644 (file)
@@ -32,7 +32,7 @@
 #include "strfsong.hxx"
 #include "mpdclient.hxx"
 #include "filelist.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "paint.hxx"
 #include "song_paint.hxx"
 
index 8dda96e..c4a276a 100644 (file)
@@ -25,7 +25,7 @@
 #include "charset.hxx"
 #include "match.hxx"
 #include "Command.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "paint.hxx"
 #include "screen_status.hxx"
 #include "screen_utils.hxx"
index fbfc539..fdb6fe3 100644 (file)
@@ -21,7 +21,7 @@
 #define LIST_WINDOW_HXX
 
 #include "config.h"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "ncmpc_curses.h"
 #include "Size.hxx"
 #include "Compiler.h"
index 02782db..d795c04 100644 (file)
@@ -18,7 +18,7 @@
  */
 
 #include "ProgressBar.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "options.hxx"
 
 #include <assert.h>
@@ -39,7 +39,7 @@ ProgressBar::Paint() const
        if (max > 0) {
                assert(width < window.size.width);
 
-               colors_use(window.w, COLOR_PROGRESSBAR);
+               SelectStyle(window.w, COLOR_PROGRESSBAR);
 
                if (width > 0)
                        mvwhline(window.w, 0, 0, '=', width);
@@ -48,12 +48,12 @@ ProgressBar::Paint() const
                unsigned x = width + 1;
 
                if (x < window.size.width) {
-                       colors_use(window.w, COLOR_PROGRESSBAR_BACKGROUND);
+                       SelectStyle(window.w, COLOR_PROGRESSBAR_BACKGROUND);
                        mvwhline(window.w, 0, x, ACS_HLINE, window.size.width - x);
                }
        } else {
                /* no progress bar, just a simple horizontal line */
-               colors_use(window.w, COLOR_LINE);
+               SelectStyle(window.w, COLOR_LINE);
                mvwhline(window.w, 0, 0, ACS_HLINE, window.size.width);
        }
 
index 4639642..74aa833 100644 (file)
@@ -20,7 +20,7 @@
 #include "StatusBar.hxx"
 #include "Event.hxx"
 #include "options.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "i18n.h"
 #include "strfsong.hxx"
 #include "player_command.hxx"
@@ -205,7 +205,7 @@ StatusBar::Paint() const
 
        wmove(w, 0, 0);
        wclrtoeol(w);
-       colors_use(w, COLOR_STATUS_BOLD);
+       SelectStyle(w, COLOR_STATUS_BOLD);
 
        if (left_text != nullptr)
                /* display state */
@@ -214,14 +214,14 @@ StatusBar::Paint() const
        if (right_width > 0) {
                /* display time string */
                int x = window.size.width - right_width;
-               colors_use(w, COLOR_STATUS_TIME);
+               SelectStyle(w, COLOR_STATUS_TIME);
                mvwaddstr(w, 0, x, right_text);
        }
 
        if (!center_text.empty()) {
                /* display song name */
 
-               colors_use(w, COLOR_STATUS);
+               SelectStyle(w, COLOR_STATUS);
 
                /* scroll if the song name is to long */
 #ifndef NCMPC_MINI
@@ -234,7 +234,7 @@ StatusBar::Paint() const
 
        /* display time string */
        int x = window.size.width - right_width;
-       colors_use(w, COLOR_STATUS_TIME);
+       SelectStyle(w, COLOR_STATUS_TIME);
        mvwaddstr(w, 0, x, right_text);
 
        wnoutrefresh(w);
@@ -268,7 +268,7 @@ StatusBar::SetMessage(const char *msg)
 
        wmove(w, 0, 0);
        wclrtoeol(w);
-       colors_use(w, COLOR_STATUS_ALERT);
+       SelectStyle(w, COLOR_STATUS_ALERT);
        waddstr(w, msg);
        wnoutrefresh(w);
 
similarity index 90%
rename from src/colors.cxx
rename to src/Styles.cxx
index 6e195b4..3d1b99c 100644 (file)
@@ -17,7 +17,7 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "BasicColors.hxx"
 #include "CustomColors.hxx"
 #include "i18n.h"
@@ -34,7 +34,7 @@
 
 #define COLOR_NONE -1
 
-struct NamedColor {
+struct StyleData {
        const char *name;
 #ifdef ENABLE_COLORS
        short fg_color;
@@ -43,12 +43,12 @@ struct NamedColor {
        attr_t mono;
 
 #ifndef ENABLE_COLORS
-       constexpr NamedColor(const char *_name, int, attr_t _mono)
+       constexpr StyleData(const char *_name, int, attr_t _mono)
                :name(_name), mono(_mono) {}
 #endif
 };
 
-static NamedColor colors[COLOR_END] = {
+static StyleData colors[COLOR_END] = {
        /* color pair = field name, color, mono */
        {nullptr, 0, 0, 0},
        {"title",             COLOR_YELLOW, A_NORMAL, A_NORMAL},
@@ -71,8 +71,8 @@ static NamedColor colors[COLOR_END] = {
 
 #ifdef ENABLE_COLORS
 
-static NamedColor *
-colors_lookup_by_name(const char *name)
+static StyleData *
+StyleByName(const char *name)
 {
        for (unsigned i = 1; i < COLOR_END; ++i)
                if (!strcasecmp(colors[i].name, name))
@@ -82,8 +82,9 @@ colors_lookup_by_name(const char *name)
 }
 
 static void
-colors_update_pair(enum color id)
+colors_update_pair(Style style)
 {
+       const size_t id(style);
        assert(id > 0 && id < COLOR_END);
 
        int fg = colors[id].fg_color;
@@ -152,9 +153,9 @@ colors_str2color(const char *str, short &fg_color, attr_t &attr)
 }
 
 bool
-colors_assign(const char *name, const char *value)
+ModifyStyle(const char *name, const char *value)
 {
-       auto *entry = colors_lookup_by_name(name);
+       auto *entry = StyleByName(name);
 
        if (!entry) {
                fprintf(stderr, "%s: %s",
@@ -166,7 +167,7 @@ colors_assign(const char *name, const char *value)
 }
 
 void
-colors_start()
+ApplyStyles()
 {
        if (has_colors()) {
                /* initialize color support */
@@ -178,7 +179,7 @@ colors_start()
                if (options.enable_colors) {
                        for (unsigned i = 1; i < COLOR_END; ++i)
                                /* update the color pairs */
-                               colors_update_pair((enum color)i);
+                               colors_update_pair(Style(i));
                }
        } else if (options.enable_colors) {
                fprintf(stderr, "%s\n",
@@ -189,12 +190,13 @@ colors_start()
 #endif
 
 void
-colors_use(WINDOW *w, enum color id)
+SelectStyle(WINDOW *w, Style style)
 {
-       auto *entry = &colors[id];
-
+       const size_t id(style);
        assert(id > 0 && id < COLOR_END);
 
+       auto *entry = &colors[size_t(style)];
+
 #ifdef ENABLE_COLORS
        if (options.enable_colors) {
                /* color mode */
similarity index 87%
rename from src/colors.hxx
rename to src/Styles.hxx
index 747b155..18ab7c3 100644 (file)
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-#ifndef COLORS_H
-#define COLORS_H
+#ifndef STYLES_HXX
+#define STYLES_HXX
 
 #include "config.h"
 #include "ncmpc_curses.h"
 
-
-enum color {
+enum Style {
        COLOR_TITLE = 1,
        COLOR_TITLE_BOLD,
        COLOR_LINE,
@@ -46,13 +45,13 @@ enum color {
 
 #ifdef ENABLE_COLORS
 bool
-colors_assign(const char *name, const char *value);
+ModifyStyle(const char *name, const char *value);
 
 void
-colors_start();
+ApplyStyles();
 #endif
 
 void
-colors_use(WINDOW *w, enum color id);
+SelectStyle(WINDOW *w, Style style);
 
-#endif /* COLORS_H */
+#endif
index 59cb1a0..f39bbb2 100644 (file)
@@ -20,7 +20,7 @@
 #include "TabBar.hxx"
 #include "PageMeta.hxx"
 #include "screen_list.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "Bindings.hxx"
 #include "GlobalBindings.hxx"
 #include "i18n.h"
 static void
 PaintPageTab(WINDOW *w, Command cmd, const char *label, bool selected)
 {
-       colors_use(w, COLOR_TITLE_BOLD);
+       SelectStyle(w, COLOR_TITLE_BOLD);
        if (selected)
                wattron(w, A_REVERSE);
 
        waddch(w, ' ');
        waddstr(w, GetGlobalKeyBindings().GetKeyNames(cmd, false));
-       colors_use(w, COLOR_TITLE);
+       SelectStyle(w, COLOR_TITLE);
        if (selected)
                wattron(w, A_REVERSE);
 
index 8463151..65f4ded 100644 (file)
@@ -19,7 +19,7 @@
 
 #include "TitleBar.hxx"
 #include "TabBar.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "options.hxx"
 #include "i18n.h"
 #include "util/StringUTF8.hxx"
@@ -88,7 +88,7 @@ TitleBar::Paint(const PageMeta &current_page_meta, const char *title) const
 #else
                (void)current_page_meta;
 #endif
-               colors_use(w, COLOR_TITLE_BOLD);
+               SelectStyle(w, COLOR_TITLE_BOLD);
                mvwaddstr(w, 0, 0, title);
 #ifndef NCMPC_MINI
        }
@@ -103,18 +103,18 @@ TitleBar::Paint(const PageMeta &current_page_meta, const char *title) const
                volume_string = buf;
        }
 
-       colors_use(w, COLOR_TITLE);
+       SelectStyle(w, COLOR_TITLE);
        mvwaddstr(w, 0, window.size.width - utf8_width(volume_string),
                  volume_string);
 
-       colors_use(w, COLOR_LINE);
+       SelectStyle(w, COLOR_LINE);
        mvwhline(w, 1, 0, ACS_HLINE, window.size.width);
        if (flags[0]) {
                wmove(w, 1, window.size.width - strlen(flags) - 3);
                waddch(w, '[');
-               colors_use(w, COLOR_LINE_FLAGS);
+               SelectStyle(w, COLOR_LINE_FLAGS);
                waddstr(w, flags);
-               colors_use(w, COLOR_LINE);
+               SelectStyle(w, COLOR_LINE);
                waddch(w, ']');
        }
 
index eeccd0b..3f7d766 100644 (file)
@@ -24,7 +24,7 @@
 #include "defaults.hxx"
 #include "i18n.h"
 #include "Command.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "BasicColors.hxx"
 #include "CustomColors.hxx"
 #include "screen_list.hxx"
@@ -255,7 +255,7 @@ parse_color(char *str)
        if (value == nullptr)
                return false;
 
-       return colors_assign(str, value);
+       return ModifyStyle(str, value);
 }
 
 /**
index 48bbaf0..906541d 100644 (file)
@@ -22,7 +22,7 @@
 #include "ncmpc_curses.h"
 
 #ifdef ENABLE_COLORS
-#include "colors.hxx"
+#include "Styles.hxx"
 #endif
 
 #ifdef HAVE_GETMOUSE
@@ -39,7 +39,7 @@ ncu_init()
 
        /* initialize color support */
 #ifdef ENABLE_COLORS
-       colors_start();
+       ApplyStyles();
 #endif
 
        /* tell curses not to do NL->CR/NL on output */
index 1ad3f54..b274222 100644 (file)
@@ -20,7 +20,7 @@
 #ifndef NCMPC_PAINT_H
 #define NCMPC_PAINT_H
 
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "options.hxx"
 
 /**
@@ -28,9 +28,9 @@
  * true.
  */
 static inline void
-row_color(WINDOW *w, enum color color, bool selected)
+row_color(WINDOW *w, Style style, bool selected)
 {
-       colors_use(w, color);
+       SelectStyle(w, style);
 
        if (selected)
                wattron(w, A_REVERSE);
@@ -67,10 +67,10 @@ row_clear_to_eol(WINDOW *w, unsigned width, bool selected)
  */
 static inline void
 row_paint_text(WINDOW *w, unsigned width,
-              enum color color, bool selected,
+              Style style, bool selected,
               const char *text)
 {
-       row_color(w, color, selected);
+       row_color(w, style, selected);
 
        waddstr(w, text);
 
index a7ff4ba..6eb9578 100644 (file)
@@ -24,7 +24,7 @@
 #include "config.h"
 #include "i18n.h"
 #include "options.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "Event.hxx"
 
 #include <stdlib.h>
index db89b9f..f00e39b 100644 (file)
@@ -23,7 +23,7 @@
 #include "config.h"
 #include "i18n.h"
 #include "options.hxx"
-#include "colors.hxx"
+#include "Styles.hxx"
 #include "wreadln.hxx"
 #include "ncmpc.hxx"
 
@@ -57,7 +57,7 @@ screen_getch(const char *prompt)
 {
        WINDOW *w = screen->status_bar.GetWindow().w;
 
-       colors_use(w, COLOR_STATUS_ALERT);
+       SelectStyle(w, COLOR_STATUS_ALERT);
        werase(w);
        wmove(w, 0, 0);
        waddstr(w, prompt);
@@ -104,7 +104,7 @@ screen_readln(const char *prompt,
 
        wmove(w, 0,0);
        curs_set(1);
-       colors_use(w, COLOR_STATUS_ALERT);
+       SelectStyle(w, COLOR_STATUS_ALERT);
        auto result = wreadln(w, prompt, value, window->size.width,
                              history, completion);
        curs_set(0);
@@ -119,7 +119,7 @@ screen_read_password(const char *prompt)
 
        wmove(w, 0,0);
        curs_set(1);
-       colors_use(w, COLOR_STATUS_ALERT);
+       SelectStyle(w, COLOR_STATUS_ALERT);
 
        if (prompt == nullptr)
                prompt = _("Password");
@@ -170,7 +170,7 @@ screen_display_completion_list(Completion::Range range)
                offset = 0;
        }
 
-       colors_use(w, COLOR_STATUS_ALERT);
+       SelectStyle(w, COLOR_STATUS_ALERT);
 
        auto i = std::next(range.begin(), offset);
        for (unsigned y = 0; y < screen->main_window.size.height; ++y, ++i) {
@@ -186,5 +186,5 @@ screen_display_completion_list(Completion::Range range)
        wclrtobot(w);
 
        wrefresh(w);
-       colors_use(w, COLOR_LIST);
+       SelectStyle(w, COLOR_LIST);
 }