From f95938c4b379da2f28b9423a682012a49a617832 Mon Sep 17 00:00:00 2001 From: Michael Leinartas Date: Mon, 12 Sep 2016 12:20:16 -0500 Subject: [PATCH] Improve if statement readability with braces --- src/utils_format_graphite.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/utils_format_graphite.c b/src/utils_format_graphite.c index 904f43c5..f09e0bc8 100644 --- a/src/utils_format_graphite.c +++ b/src/utils_format_graphite.c @@ -150,33 +150,28 @@ static int gr_format_name (char *ret, int ret_len, sstrncpy (tmp_plugin, n_plugin, sizeof (tmp_plugin)); if (n_type_instance[0] != '\0') + { if ((flags & GRAPHITE_DROP_DUPE_FIELDS) && strcmp(n_plugin, n_type) == 0) - { sstrncpy (tmp_type, n_type_instance, sizeof (tmp_type)); - } else - { ssnprintf (tmp_type, sizeof (tmp_type), "%s%c%s", n_type, (flags & GRAPHITE_SEPARATE_INSTANCES) ? '.' : '-', n_type_instance); - } + } else sstrncpy (tmp_type, n_type, sizeof (tmp_type)); - /* Assert always_append_ds -> ds_name */ assert (!(flags & GRAPHITE_ALWAYS_APPEND_DS) || (ds_name != NULL)); if (ds_name != NULL) + { if ((flags & GRAPHITE_DROP_DUPE_FIELDS) && strcmp(tmp_plugin, tmp_type) == 0) - { ssnprintf (ret, ret_len, "%s%s%s.%s.%s", prefix, n_host, postfix, tmp_plugin, ds_name); - } else - { ssnprintf (ret, ret_len, "%s%s%s.%s.%s.%s", prefix, n_host, postfix, tmp_plugin, tmp_type, ds_name); - } + } else ssnprintf (ret, ret_len, "%s%s%s.%s.%s", prefix, n_host, postfix, tmp_plugin, tmp_type); -- 2.30.2