diff options
author | Aleksander Morgado <aleksander@lanedo.com> | 2011-12-01 19:41:18 +0100 |
---|---|---|
committer | Aleksander Morgado <aleksander@lanedo.com> | 2012-03-15 14:14:31 +0100 |
commit | 0c64e08f853ad01cd4e459c4f57e9d8469b20286 (patch) | |
tree | 0674f2860c319f668b2dae2ec26eb420a344c5f8 /src | |
parent | fb90f205d1345a9dd3f9a3007a534f043a32952d (diff) |
error-helpers: small refactor
Diffstat (limited to 'src')
-rw-r--r-- | src/mm-error-helpers.c | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/src/mm-error-helpers.c b/src/mm-error-helpers.c index 67ae7f8c..06eb4ed9 100644 --- a/src/mm-error-helpers.c +++ b/src/mm-error-helpers.c @@ -116,25 +116,21 @@ static ErrorTable me_errors[] = { GError * mm_mobile_equipment_error_for_code (MMMobileEquipmentError code) { - const gchar *msg = NULL; guint i; /* Look for the code */ for (i = 0; i < G_N_ELEMENTS (me_errors); i++) { - if (me_errors[i].code == code) { - msg = me_errors[i].message; - break; - } + if (me_errors[i].code == code) + return g_error_new_literal (MM_MOBILE_EQUIPMENT_ERROR, + code, + me_errors[i].message); } /* Not found? Then, default */ - if (!msg) { - g_warning ("Invalid mobile equipment error code: %d", code); - code = MM_MOBILE_EQUIPMENT_ERROR_UNKNOWN; - msg = "Unknown error"; - } - - return g_error_new_literal (MM_MOBILE_EQUIPMENT_ERROR, code, msg); + g_warning ("Invalid mobile equipment error code: %u", (guint)code); + return g_error_new (MM_MOBILE_EQUIPMENT_ERROR, + MM_MOBILE_EQUIPMENT_ERROR_UNKNOWN, + "Unknown error"); } GError * |