diff options
author | Kalev Lember <kalevlember@gmail.com> | 2013-04-03 00:35:27 +0200 |
---|---|---|
committer | Kalev Lember <kalevlember@gmail.com> | 2013-05-07 13:59:53 +0200 |
commit | 19e6db7997106bb2485b3982fb625ed964e19b3c (patch) | |
tree | 1d648fa034f81760e53a72c1c5079bfdac4e8086 /gtk/gtksizegroup.c | |
parent | 1391a0be1cbcd579628c9beaafea47e13bb7df5b (diff) | |
download | gtk+-19e6db7997106bb2485b3982fb625ed964e19b3c.tar.gz |
gtksizegroup: Add explicit braces to avoid dangling else
Otherwise the else after the for loop becomes attached to the wrong if.
Diffstat (limited to 'gtk/gtksizegroup.c')
-rw-r--r-- | gtk/gtksizegroup.c | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/gtk/gtksizegroup.c b/gtk/gtksizegroup.c index e31dfa84eb..f84ed47f48 100644 --- a/gtk/gtksizegroup.c +++ b/gtk/gtksizegroup.c @@ -667,15 +667,22 @@ size_group_start_element (GMarkupParseContext *context, GSListSubParserData *data = (GSListSubParserData*)user_data; if (strcmp (element_name, "widget") == 0) - for (i = 0; names[i]; i++) - if (strcmp (names[i], "name") == 0) - data->items = g_slist_prepend (data->items, g_strdup (values[i])); + { + for (i = 0; names[i]; i++) + { + if (strcmp (names[i], "name") == 0) + data->items = g_slist_prepend (data->items, g_strdup (values[i])); + } + } else if (strcmp (element_name, "widgets") == 0) - return; + { + return; + } else - g_warning ("Unsupported type tag for GtkSizeGroup: %s\n", - element_name); - + { + g_warning ("Unsupported type tag for GtkSizeGroup: %s\n", + element_name); + } } static const GMarkupParser size_group_parser = |