From b6c4d4ee4602fc34d0e9049bd61d0ac4babb78bf Mon Sep 17 00:00:00 2001 From: Michael Stapelberg Date: Tue, 26 Feb 2013 00:32:24 +0100 Subject: [PATCH] wireless: call START_COLOR only once (Thanks XTaran) --- src/print_wireless_info.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/print_wireless_info.c b/src/print_wireless_info.c index a678f52..18f6580 100644 --- a/src/print_wireless_info.c +++ b/src/print_wireless_info.c @@ -332,16 +332,15 @@ void print_wireless_info(yajl_gen json_gen, char *buffer, const char *interface, START_COLOR("color_bad"); outwalk += sprintf(outwalk, "%s", format_down); goto out; - } else { - START_COLOR("color_good"); } if (get_wireless_info(interface, &info)) { walk = format_up; if (info.flags & WIRELESS_INFO_FLAG_HAS_QUALITY) START_COLOR((info.quality < info.quality_average ? "color_degraded" : "color_good")); - } - else { + else + START_COLOR("color_good"); + } else { walk = format_down; START_COLOR("color_bad"); } -- 2.39.5