diff options
| author | Ben Chan <benchan@chromium.org> | 2018-10-17 23:06:49 -0700 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2018-10-18 13:17:36 +0200 |
| commit | 436f9ef7e6d5f04ef1c01e595831a7ae9a4f2cbf (patch) | |
| tree | d0b1d09b1722b96187180d6a266c3af35ab4da6b /src/mm-base-sms.c | |
| parent | 7de6b6f2e176ed60f1f74d0d8704f73e39585891 (diff) | |
core: use g_clear_pointer() to avoid explicitly checking and resetting pointers
Diffstat (limited to 'src/mm-base-sms.c')
| -rw-r--r-- | src/mm-base-sms.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/src/mm-base-sms.c b/src/mm-base-sms.c index e117bef7..25a02617 100644 --- a/src/mm-base-sms.c +++ b/src/mm-base-sms.c @@ -939,10 +939,7 @@ sms_store_next_part (GTask *task) return; } - if (ctx->msg_data) { - g_free (ctx->msg_data); - ctx->msg_data = NULL; - } + g_clear_pointer (&ctx->msg_data, g_free); if (!sms_get_store_or_send_command ((MMSmsPart *)ctx->current->data, ctx->use_pdu_mode, @@ -1218,10 +1215,7 @@ sms_send_next_part (GTask *task) /* Generic send */ - if (ctx->msg_data) { - g_free (ctx->msg_data); - ctx->msg_data = NULL; - } + g_clear_pointer (&ctx->msg_data, g_free); if (!sms_get_store_or_send_command ((MMSmsPart *)ctx->current->data, ctx->use_pdu_mode, |