diff options
| author | Ben Chan <benchan@chromium.org> | 2018-10-17 23:06:48 -0700 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2018-10-18 13:17:36 +0200 |
| commit | 7de6b6f2e176ed60f1f74d0d8704f73e39585891 (patch) | |
| tree | e7a8e4adb2c7b6cd9da12be4c281440f6a796bc6 /plugins/xmm/mm-modem-helpers-xmm.c | |
| parent | 749b806c30b41512dec4901808976425cdf3e56b (diff) | |
core: remove unnecessary NULL checks for g_match_info_free()
g_match_info_free() already check if the given pointer is NULL and does
nothing on a NULL pointer.
Diffstat (limited to 'plugins/xmm/mm-modem-helpers-xmm.c')
| -rw-r--r-- | plugins/xmm/mm-modem-helpers-xmm.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/plugins/xmm/mm-modem-helpers-xmm.c b/plugins/xmm/mm-modem-helpers-xmm.c index a3eb5f9b..4ccbc745 100644 --- a/plugins/xmm/mm-modem-helpers-xmm.c +++ b/plugins/xmm/mm-modem-helpers-xmm.c @@ -449,8 +449,7 @@ mm_xmm_parse_xact_query_response (const gchar *response, /* success */ out: - if (match_info) - g_match_info_free (match_info); + g_match_info_free (match_info); g_regex_unref (r); if (inner_error) { @@ -672,9 +671,7 @@ mm_xmm_parse_xcesq_query_response (const gchar *response, } out: - - if (match_info) - g_match_info_free (match_info); + g_match_info_free (match_info); g_regex_unref (r); if (inner_error) { @@ -996,8 +993,7 @@ mm_xmm_parse_xlcsslp_query_response (const gchar *response, } out: - if (match_info) - g_match_info_free (match_info); + g_match_info_free (match_info); g_regex_unref (r); if (inner_error) { |