diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2020-11-22 15:05:56 +0100 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2021-02-23 11:35:11 +0000 |
| commit | 8b590721dfdff5c4382dbc3498d92035b2d4ad64 (patch) | |
| tree | 0215270cf248ec39e0be4760a5eaf99487f358be /src/mm-sms-part-cdma.c | |
| parent | 38a4a9c842bcb0c87bca7b1a5595556fe2c144fc (diff) | |
charsets: don't allow quoting in byte_array_append()
There's no point in adding a quoting option to this method; if the
caller needs the appended string quoted, it should quote it before
passing it to this method.
It was nowhere used anyway.
Diffstat (limited to 'src/mm-sms-part-cdma.c')
| -rw-r--r-- | src/mm-sms-part-cdma.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/src/mm-sms-part-cdma.c b/src/mm-sms-part-cdma.c index e08193cf..fcfd85e9 100644 --- a/src/mm-sms-part-cdma.c +++ b/src/mm-sms-part-cdma.c @@ -1411,11 +1411,7 @@ decide_best_encoding (const gchar *text, /* Check if we can do Latin encoding */ if (mm_charset_can_convert_to (text, MM_MODEM_CHARSET_8859_1)) { *out = g_byte_array_sized_new (len); - if (!mm_modem_charset_byte_array_append (*out, - text, - FALSE, - MM_MODEM_CHARSET_8859_1, - &error)) + if (!mm_modem_charset_byte_array_append (*out, text, MM_MODEM_CHARSET_8859_1, &error)) mm_obj_warn (log_object, "failed to convert to latin encoding: %s", error->message); *num_fields = (*out)->len; *num_bits_per_field = 8; @@ -1425,11 +1421,7 @@ decide_best_encoding (const gchar *text, /* If no Latin and no ASCII, default to UTF-16 */ *out = g_byte_array_sized_new (len * 2); - if (!mm_modem_charset_byte_array_append (*out, - text, - FALSE, - MM_MODEM_CHARSET_UCS2, - &error)) + if (!mm_modem_charset_byte_array_append (*out, text, MM_MODEM_CHARSET_UCS2, &error)) mm_obj_warn (log_object, "failed to convert to UTF-16 encoding: %s", error->message); *num_fields = (*out)->len / 2; *num_bits_per_field = 16; |