From 02ba0604909f842b289f760c9b73a8451a1a8a86 Mon Sep 17 00:00:00 2001 From: Benedikt Heine Date: Thu, 27 Jul 2017 20:34:03 +0200 Subject: [PATCH] Refactor if chain to avoid nested statements --- src/dbus.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/dbus.c b/src/dbus.c index 27acc3c..4894b83 100644 --- a/src/dbus.c +++ b/src/dbus.c @@ -216,14 +216,10 @@ static void on_notify(GDBusConnection * connection, else if((dict_value = g_variant_lookup_value(content, "transient", G_VARIANT_TYPE_INT32))) transient = g_variant_get_int32(dict_value) > 0; - dict_value = g_variant_lookup_value(content, "value", G_VARIANT_TYPE_INT32); - if (dict_value) { + if((dict_value = g_variant_lookup_value(content, "value", G_VARIANT_TYPE_INT32))) progress = g_variant_get_int32(dict_value); - } else { - dict_value = g_variant_lookup_value(content, "value", G_VARIANT_TYPE_UINT32); - if (dict_value) - progress = g_variant_get_uint32(dict_value); - } + else if((dict_value = g_variant_lookup_value(content, "value", G_VARIANT_TYPE_UINT32))) + progress = g_variant_get_uint32(dict_value); } break; case 7: