From: Michael Leinartas <mleinartas@twitter.com>
Date: Mon, 12 Sep 2016 17:20:16 +0000 (-0500)
Subject: Improve if statement readability with braces
X-Git-Tag: collectd-5.7.0~100^2~1
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=f95938c4b379da2f28b9423a682012a49a617832;p=collectd.git

Improve if statement readability with braces
---

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);