diff options
author | Denis Kenzior <denkenz@gmail.com> | 2011-11-16 03:59:40 -0600 |
---|---|---|
committer | Denis Kenzior <denkenz@gmail.com> | 2011-11-16 03:59:40 -0600 |
commit | f454dea15bdb5b69c136f40405295258739f20fc (patch) | |
tree | f160c7c733f0d7228929d03226f922e4a1bf578f /plugins/mbpi.c | |
parent | 07c33cdee9445a07ae2e2c8b751f8603c3e4edbb (diff) | |
download | ofono-f454dea15bdb5b69c136f40405295258739f20fc.tar.bz2 |
plugins: Make the code a bit prettier
Diffstat (limited to 'plugins/mbpi.c')
-rw-r--r-- | plugins/mbpi.c | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/plugins/mbpi.c b/plugins/mbpi.c index 415633f0..309e1ed4 100644 --- a/plugins/mbpi.c +++ b/plugins/mbpi.c @@ -296,11 +296,13 @@ static void sid_handler(GMarkupParseContext *context, const char *sid = NULL; int i; - for (i = 0; attribute_names[i]; i++) - if (g_str_equal(attribute_names[i], "value") == TRUE) { - sid = attribute_values[i]; - break; - } + for (i = 0; attribute_names[i]; i++) { + if (g_str_equal(attribute_names[i], "value") == FALSE) + continue; + + sid = attribute_values[i]; + break; + } if (sid == NULL) { mbpi_g_set_error(context, error, G_MARKUP_ERROR, @@ -484,14 +486,13 @@ static void toplevel_cdma_start(GMarkupParseContext *context, { struct cdma_data *cdma = userdata; - if (g_str_equal(element_name, "provider")) { - if (cdma->match_found == TRUE) - g_markup_parse_context_push(context, &skip_parser, - NULL); - else - g_markup_parse_context_push(context, &provider_parser, - userdata); - } + if (g_str_equal(element_name, "provider") == FALSE) + return; + + if (cdma->match_found == TRUE) + g_markup_parse_context_push(context, &skip_parser, NULL); + else + g_markup_parse_context_push(context, &provider_parser, cdma); } static void toplevel_cdma_end(GMarkupParseContext *context, |