diff options
| author | Ben Chan <benchan@chromium.org> | 2012-08-25 21:34:55 -0700 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@lanedo.com> | 2012-08-27 11:17:30 +0200 |
| commit | 178de0e51d925f7b20aedcfc6b773e9d173d13a8 (patch) | |
| tree | 39c69ca64b12b6381fd94af2f06708e97fc60ee3 | |
| parent | e997803594873dffbdd9922e2f4f682f07e9a0eb (diff) | |
sms-part: use g_set_error instead of g_error_new for consistency
| -rw-r--r-- | src/mm-sms-part.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/mm-sms-part.c b/src/mm-sms-part.c index 6637325a..53b66785 100644 --- a/src/mm-sms-part.c +++ b/src/mm-sms-part.c @@ -440,11 +440,12 @@ mm_sms_part_new_from_pdu (guint index, sender_addr_num_octets = (sender_addr_num_digits + 1) >> 1; variable_length_items += sender_addr_num_octets; if (pdu_len < variable_length_items + SMS_MIN_PDU_LEN) { - *error = g_error_new (MM_CORE_ERROR, - MM_CORE_ERROR_FAILED, - "PDU too short (2): %zd < %d", - pdu_len, - variable_length_items + SMS_MIN_PDU_LEN); + g_set_error (error, + MM_CORE_ERROR, + MM_CORE_ERROR_FAILED, + "PDU too short (2): %zd < %d", + pdu_len, + variable_length_items + SMS_MIN_PDU_LEN); g_free (pdu); return NULL; } |