From 5f91a400685c0762df5f0f3bbb0b97daaf0d2371 Mon Sep 17 00:00:00 2001 From: Holger Hans Peter Freyther Date: Fri, 5 Dec 2014 10:29:45 +0100 Subject: logging: Call color only once and use the result First we check if a color is defined and then we call it again and use the result. Avoid the second call and use the result of the previous call. --- src/logging.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/logging.c b/src/logging.c index 7b532779..36c31c4a 100644 --- a/src/logging.c +++ b/src/logging.c @@ -245,7 +245,7 @@ static void _output(struct log_target *target, unsigned int subsys, if (target->use_color) { const char *c = color(subsys); if (c) { - ret = snprintf(buf + offset, rem, "%s", color(subsys)); + ret = snprintf(buf + offset, rem, "%s", c); if (ret < 0) goto err; OSMO_SNPRINTF_RET(ret, rem, offset, len); -- cgit v1.2.3