From 05a1097a27ff7c7349e6d9730a7009c3b6d9670b Mon Sep 17 00:00:00 2001 From: Ben Chan Date: Mon, 7 Aug 2017 14:05:03 -0700 Subject: iface-modem: remove unnecessary error check This patch removes an unnecessary error check in the update_unlock_retries() where the error is never set. --- src/mm-iface-modem.c | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) (limited to 'src/mm-iface-modem.c') diff --git a/src/mm-iface-modem.c b/src/mm-iface-modem.c index ec453a8f..c025b5c5 100644 --- a/src/mm-iface-modem.c +++ b/src/mm-iface-modem.c @@ -2931,7 +2931,6 @@ update_unlock_retries (MMIfaceModem *self, MMUnlockRetries *unlock_retries) { MmGdbusModem *skeleton = NULL; - GError *error = NULL; GVariant *previous_dictionary; MMUnlockRetries *previous_unlock_retries; @@ -2944,18 +2943,13 @@ update_unlock_retries (MMIfaceModem *self, previous_dictionary = mm_gdbus_modem_get_unlock_retries (skeleton); previous_unlock_retries = mm_unlock_retries_new_from_dictionary (previous_dictionary); - if (error) { - mm_warn ("Couldn't build previous unlock retries: '%s'", error->message); - g_error_free (error); - } else { - /* If they are different, update */ - if (!mm_unlock_retries_cmp (unlock_retries, previous_unlock_retries)) { - GVariant *new_dictionary; + /* If they are different, update */ + if (!mm_unlock_retries_cmp (unlock_retries, previous_unlock_retries)) { + GVariant *new_dictionary; - new_dictionary = mm_unlock_retries_get_dictionary (unlock_retries); - mm_gdbus_modem_set_unlock_retries (skeleton, new_dictionary); - g_variant_unref (new_dictionary); - } + new_dictionary = mm_unlock_retries_get_dictionary (unlock_retries); + mm_gdbus_modem_set_unlock_retries (skeleton, new_dictionary); + g_variant_unref (new_dictionary); } g_object_unref (previous_unlock_retries); -- cgit v1.2.3-70-g09d2