Removed unused variable left over from my previous fix.
2003-03-17 Jeffrey Stedfast <fejj@ximian.com> * 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
This commit is contained in:
committed by
Jeffrey Stedfast
parent
e1328c5de8
commit
8148ebb2d9
@ -1,3 +1,14 @@
|
||||
2003-03-17 Jeffrey Stedfast <fejj@ximian.com>
|
||||
|
||||
* 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 <fejj@ximian.com>
|
||||
|
||||
* camel-mime-part.c (process_header): Use
|
||||
|
||||
@ -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 */
|
||||
|
||||
@ -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");
|
||||
|
||||
Reference in New Issue
Block a user