[evolution] Postscript attachments not shown in message preview
- From: Milan Crha <mcrha src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution] Postscript attachments not shown in message preview
- Date: Mon, 9 Feb 2015 17:10:56 +0000 (UTC)
commit a4af2d6aeefa48bf8a55cd3dcc11578983fa3cd2
Author: Milan Crha <mcrha redhat com>
Date: Mon Feb 9 18:08:29 2015 +0100
Postscript attachments not shown in message preview
The application/postscript mime type had been handled by the text-hightlight
module, but thinking of it, it doesn't make much sense, because most users
would prefer to see the content rather in a postscript viewer, than
the source of the postscript file, thus this change removes the mime type
from the text-hightlight module. There had been also issues about thinking
that certain parts had been handled, while they did not, which this change
fixes as well.
The issue had been reported downstream at:
https://bugzilla.redhat.com/show_bug.cgi?id=1187327
em-format/e-mail-parser-text-plain.c | 7 ++++---
.../text-highlight/e-mail-parser-text-highlight.c | 5 +++--
modules/text-highlight/languages.c | 5 -----
3 files changed, 7 insertions(+), 10 deletions(-)
---
diff --git a/em-format/e-mail-parser-text-plain.c b/em-format/e-mail-parser-text-plain.c
index 7fbc59c..ef3e4f7 100644
--- a/em-format/e-mail-parser-text-plain.c
+++ b/em-format/e-mail-parser-text-plain.c
@@ -56,10 +56,11 @@ process_part (EMailParser *parser,
CamelContentType *type;
EMailPart *mail_part;
gint s_len = part_id->len;
+ gboolean handled = TRUE;
type = camel_mime_part_get_content_type (part);
if (!camel_content_type_is (type, "text", "*")) {
- e_mail_parser_parse_part (
+ handled = e_mail_parser_parse_part (
parser, CAMEL_MIME_PART (part), part_id,
cancellable, out_mail_parts);
@@ -88,14 +89,14 @@ process_part (EMailParser *parser,
} else {
g_string_append_printf (part_id, ".inline.%d", part_number);
- e_mail_parser_parse_part (
+ handled = e_mail_parser_parse_part (
parser, CAMEL_MIME_PART (part), part_id,
cancellable, out_mail_parts);
g_string_truncate (part_id, s_len);
}
- return TRUE;
+ return handled;
}
static gboolean
diff --git a/modules/text-highlight/e-mail-parser-text-highlight.c
b/modules/text-highlight/e-mail-parser-text-highlight.c
index d729a38..e27c2b3 100644
--- a/modules/text-highlight/e-mail-parser-text-highlight.c
+++ b/modules/text-highlight/e-mail-parser-text-highlight.c
@@ -55,6 +55,7 @@ empe_text_highlight_parse (EMailParserExtension *extension,
GQueue *out_mail_parts)
{
CamelContentType *ct;
+ gboolean handled;
gint len;
/* Prevent recursion */
@@ -77,13 +78,13 @@ empe_text_highlight_parse (EMailParserExtension *extension,
/* All source codes and scripts are in general plain texts,
* so let text/plain parser handle it. */
- e_mail_parser_parse_part_as (
+ handled = e_mail_parser_parse_part_as (
parser, part, part_id, "text/plain",
cancellable, out_mail_parts);
g_string_truncate (part_id, len);
- return TRUE;
+ return handled;
}
static void
diff --git a/modules/text-highlight/languages.c b/modules/text-highlight/languages.c
index a554a4c..ef232a0 100644
--- a/modules/text-highlight/languages.c
+++ b/modules/text-highlight/languages.c
@@ -320,11 +320,6 @@ static struct Language other_languages[] = {
(const gchar *[]) { NULL }
},
- { "ps", N_("_PostScript"),
- (const gchar *[]) { (gchar[]) { "ps" } , NULL },
- (const gchar *[]) { (gchar[]) { "application/postscript" }, NULL }
- },
-
{ "r", N_("_R"),
(const gchar *[]) { (gchar[]) { "r" }, NULL },
(const gchar *[]) { NULL }
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]