Make the prettyping output work with and without colors.

[liboping.git] / src / oping.c

diff --git a/src/oping.c b/src/oping.c
index a19c420..5c0ead5 100644 (file)
--- a/src/oping.c
+++ b/src/oping.c
@@ -81,7 +81,12 @@
 # define NCURSES_OPAQUE 1
 /* http://newsgroups.derkeiler.com/Archive/Rec/rec.games.roguelike.development/2010-09/msg00050.html */
 # define _X_OPEN_SOURCE_EXTENDED
-# include <ncursesw/ncurses.h>
+
+# if HAVE_NCURSESW_NCURSES_H
+#  include <ncursesw/ncurses.h>
+# elif HAVE_NCURSES_H
+#  include <ncurses.h>
+# endif
 
 # define OPING_GREEN 1
 # define OPING_YELLOW 2
@@ -652,6 +657,7 @@ static void time_calc (struct timespec *ts_dest, /* {{{ */
 #if USE_NCURSES
 static _Bool has_utf8() /* {{{ */
 {
+# if HAVE_NCURSESW_NCURSES_H
        if (!opt_utf8)
        {
                /* Automatically determine */
@@ -661,6 +667,9 @@ static _Bool has_utf8() /* {{{ */
                        opt_utf8 = 1;
        }
        return ((_Bool) (opt_utf8 - 1));
+# else
+       return (0);
+# endif
 } /* }}} _Bool has_utf8 */
 
 static int update_prettyping_graph (ping_context_t *ctx, /* {{{ */
@@ -669,8 +678,6 @@ static int update_prettyping_graph (ping_context_t *ctx, /* {{{ */
        int color = OPING_RED;
        char const *symbol = "!";
        int symbolc = '!';
-       size_t hist_symbols_num;
-       size_t index_symbols;
 
        int x_max;
        int x_pos;
@@ -678,32 +685,39 @@ static int update_prettyping_graph (ping_context_t *ctx, /* {{{ */
        x_max = getmaxx (ctx->window);
        x_pos = ((sequence - 1) % (x_max - 4)) + 2;
 
-       if (has_utf8())
-       {
-               hist_symbols_num = hist_symbols_utf8_num;
-       }
-       else {
-               hist_symbols_num = hist_symbols_acs_num;
-       }
-
        if (latency >= 0.0)
        {
                double ratio;
-               size_t intensity;
+
+               size_t symbols_num = hist_symbols_acs_num;
+               size_t colors_num = 1;
+
+               size_t index_symbols;
                size_t index_colors;
+               size_t intensity;
 
-               ratio = latency / PING_DEF_TTL;
+               /* latency is in milliseconds, opt_interval is in seconds. */
+               ratio = (latency * 0.001) / opt_interval;
                if (ratio > 1) {
                        ratio = 1.0;
                }
 
-               intensity = (size_t) ((ratio * hist_symbols_num
-                                       * hist_colors_num) - 1);
+               if (has_utf8 ())
+                       symbols_num = hist_symbols_utf8_num;
 
-               index_colors = intensity / hist_symbols_num;
-               assert (index_colors < hist_colors_num);
+               if (has_colors () == TRUE)
+                       colors_num = hist_colors_num;
+
+               intensity = (size_t) (ratio * ((double) (symbols_num * colors_num)));
+               if (intensity >= (symbols_num * colors_num))
+                       intensity = (symbols_num * colors_num) - 1;
+
+               index_symbols = intensity % symbols_num;
+               assert (index_symbols < symbols_num);
+
+               index_colors = intensity / symbols_num;
+               assert (index_colors < colors_num);
 
-               index_symbols = intensity % hist_symbols_num;
                if (has_utf8())
                {
                        color = hist_colors_utf8[index_colors];
@@ -714,25 +728,20 @@ static int update_prettyping_graph (ping_context_t *ctx, /* {{{ */
                        color = hist_colors_acs[index_colors];
                        symbolc = hist_symbols_acs[index_symbols] | A_ALTCHARSET;
                }
-        }
+       }
        else /* if (!(latency >= 0.0)) */
                wattron (ctx->window, A_BOLD);
 
-       wattron (ctx->window, COLOR_PAIR(color));
+       if (has_colors () == TRUE)
+               wattron (ctx->window, COLOR_PAIR(color));
+
        if (has_utf8())
-       {
-               mvwprintw (ctx->window,
-                          /* y = */ 3,
-                          /* x = */ x_pos,
-                          symbol);
-       }
-       else {
-               mvwaddch (ctx->window,
-                         /* y = */ 3,
-                         /* x = */ x_pos,
-                         symbolc);
-       }
-       wattroff (ctx->window, COLOR_PAIR(color));
+               mvwprintw (ctx->window, /* y = */ 3, /* x = */ x_pos, symbol);
+       else
+               mvwaddch (ctx->window, /* y = */ 3, /* x = */ x_pos, symbolc);
+
+       if (has_colors () == TRUE)
+               wattroff (ctx->window, COLOR_PAIR(color));
 
        /* Use negation here to handle NaN correctly. */
        if (!(latency >= 0.0))
@@ -789,8 +798,7 @@ static int update_stats_from_context (ping_context_t *ctx, pingobj_iter_t *iter)
                                deviation);
        }
 
-       if (has_colors () == TRUE)
-               update_prettyping_graph (ctx, latency, sequence);
+       update_prettyping_graph (ctx, latency, sequence);
 
        wrefresh (ctx->window);