From 8148ebb2d9d8716d9ad341d3ac709a26eea123f2 Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Mon, 17 Mar 2003 16:53:57 +0000 Subject: Removed unused variable left over from my previous fix. 2003-03-17 Jeffrey Stedfast * camel-mime-part.c (process_header): Removed unused variable left over from my previous fix. * providers/smtp/camel-smtp-transport.c (smtp_send_to): Don't pass 'has_8bit_parts' to smtp_data() anymore. (smtp_data): No longer takes 'has_8bit_parts' argument. Ignore whether or not the message has 8bit parts when deciding what the required encoding type we need to enforce. Fixes bug #39744. svn path=/trunk/; revision=20312 --- camel/ChangeLog | 11 +++++++++++ camel/camel-mime-part.c | 2 +- camel/providers/smtp/camel-smtp-transport.c | 22 ++++++++-------------- 3 files changed, 20 insertions(+), 15 deletions(-) (limited to 'camel') diff --git a/camel/ChangeLog b/camel/ChangeLog index c94f27a563..6ffac9feee 100644 --- a/camel/ChangeLog +++ b/camel/ChangeLog @@ -1,3 +1,14 @@ +2003-03-17 Jeffrey Stedfast + + * camel-mime-part.c (process_header): Removed unused variable left + over from my previous fix. + + * providers/smtp/camel-smtp-transport.c (smtp_send_to): Don't pass + 'has_8bit_parts' to smtp_data() anymore. + (smtp_data): No longer takes 'has_8bit_parts' argument. Ignore + whether or not the message has 8bit parts when deciding what the + required encoding type we need to enforce. Fixes bug #39744. + 2003-03-13 Jeffrey Stedfast * camel-mime-part.c (process_header): Use diff --git a/camel/camel-mime-part.c b/camel/camel-mime-part.c index ad09728c5f..0a696ed1e9 100644 --- a/camel/camel-mime-part.c +++ b/camel/camel-mime-part.c @@ -211,7 +211,7 @@ process_header(CamelMedium *medium, const char *header_name, const char *header_ { CamelMimePart *mime_part = CAMEL_MIME_PART (medium); CamelHeaderType header_type; - const char *charset, *p; + const char *charset; char *text; /* Try to parse the header pair. If it corresponds to something */ diff --git a/camel/providers/smtp/camel-smtp-transport.c b/camel/providers/smtp/camel-smtp-transport.c index 57e1b52779..86caa635dc 100644 --- a/camel/providers/smtp/camel-smtp-transport.c +++ b/camel/providers/smtp/camel-smtp-transport.c @@ -81,8 +81,7 @@ static gboolean smtp_auth (CamelSmtpTransport *transport, const char *mech, Came static gboolean smtp_mail (CamelSmtpTransport *transport, const char *sender, gboolean has_8bit_parts, CamelException *ex); static gboolean smtp_rcpt (CamelSmtpTransport *transport, const char *recipient, CamelException *ex); -static gboolean smtp_data (CamelSmtpTransport *transport, CamelMimeMessage *message, - gboolean has_8bit_parts, CamelException *ex); +static gboolean smtp_data (CamelSmtpTransport *transport, CamelMimeMessage *message, CamelException *ex); static gboolean smtp_rset (CamelSmtpTransport *transport, CamelException *ex); static gboolean smtp_quit (CamelSmtpTransport *transport, CamelException *ex); @@ -722,10 +721,7 @@ smtp_send_to (CamelTransport *transport, CamelMimeMessage *message, } } - /* passing in has_8bit_parts saves time as we don't have to - recurse through the message all over again if the user is - not sending 8bit mime parts */ - if (!smtp_data (smtp_transport, message, has_8bit_parts, ex)) { + if (!smtp_data (smtp_transport, message, ex)) { camel_operation_end (NULL); return FALSE; } @@ -1222,7 +1218,7 @@ smtp_rcpt (CamelSmtpTransport *transport, const char *recipient, CamelException } static gboolean -smtp_data (CamelSmtpTransport *transport, CamelMimeMessage *message, gboolean has_8bit_parts, CamelException *ex) +smtp_data (CamelSmtpTransport *transport, CamelMimeMessage *message, CamelException *ex) { CamelBestencEncoding enctype = CAMEL_BESTENC_8BIT; struct _header_raw *header, *savedbcc, *n, *tail; @@ -1231,16 +1227,14 @@ smtp_data (CamelSmtpTransport *transport, CamelMimeMessage *message, gboolean ha CamelMimeFilter *crlffilter; int ret; - /* if the message contains 8bit/binary mime parts and the server - doesn't support it, set our required encoding to be 7bit */ - if (has_8bit_parts && !(transport->flags & CAMEL_SMTP_TRANSPORT_8BITMIME)) + /* If the server doesn't support 8BITMIME, set our required encoding to be 7bit */ + if (!(transport->flags & CAMEL_SMTP_TRANSPORT_8BITMIME)) enctype = CAMEL_BESTENC_7BIT; /* FIXME: should we get the best charset too?? */ - /* Changes the encoding of any 8bit/binary mime parts to fit - within our required encoding type and also force any text - parts with long lines (longer than 998 octets) to wrap by - QP or base64 encoding them. */ + /* Changes the encoding of all mime parts to fit within our required + encoding type and also force any text parts with long lines (longer + than 998 octets) to wrap by QP or base64 encoding them. */ camel_mime_message_set_best_encoding (message, CAMEL_BESTENC_GET_ENCODING, enctype); cmdbuf = g_strdup ("DATA\r\n"); -- cgit