diff options
| author | Jeffrey Stedfast <fejj@helixcode.com> | 2000-10-07 01:59:39 +0800 |
|---|---|---|
| committer | Jeffrey Stedfast <fejj@src.gnome.org> | 2000-10-07 01:59:39 +0800 |
| commit | 734f0a9402b4f7f66a36281fb577f3e47eaaa574 (patch) | |
| tree | ba92f996e3ada8d9227bc618311670446944a4a5 | |
| parent | 5ef9ff4315fc314e2f9e930d1631ee5ea6b9a97c (diff) | |
| download | gsoc2013-evolution-734f0a9402b4f7f66a36281fb577f3e47eaaa574.tar.gz gsoc2013-evolution-734f0a9402b4f7f66a36281fb577f3e47eaaa574.tar.zst gsoc2013-evolution-734f0a9402b4f7f66a36281fb577f3e47eaaa574.zip | |
Fix so that we don't encode every single char in the word. Also, do we
2000-10-06 Jeffrey Stedfast <fejj@helixcode.com>
* camel-mime-utils.c (quoted_encode): Fix so that we don't encode
every single char in the word. Also, do we need a safemask? I
don't see why we would.
svn path=/trunk/; revision=5765
| -rw-r--r-- | camel/ChangeLog | 6 | ||||
| -rw-r--r-- | camel/camel-mime-utils.c | 25 |
2 files changed, 19 insertions, 12 deletions
diff --git a/camel/ChangeLog b/camel/ChangeLog index a6df9552a8..37ba9b0de7 100644 --- a/camel/ChangeLog +++ b/camel/ChangeLog @@ -1,3 +1,9 @@ +2000-10-06 Jeffrey Stedfast <fejj@helixcode.com> + + * camel-mime-utils.c (quoted_encode): Fix so that we don't encode + every single char in the word. Also, do we need a safemask? I + don't see why we would. + 2000-10-06 Chris Toshok <toshok@helixcode.com> * providers/nntp/Makefile.am (libcamelnntpinclude_HEADERS): add diff --git a/camel/camel-mime-utils.c b/camel/camel-mime-utils.c index d8f10672ac..3ddc8abf74 100644 --- a/camel/camel-mime-utils.c +++ b/camel/camel-mime-utils.c @@ -789,6 +789,7 @@ quoted_decode(const unsigned char *in, int len, unsigned char *out) /* rfc2047 version of quoted-printable */ /* safemask is the mask to apply to the camel_mime_special_table to determine what characters can safely be included without encoding */ +/* Why do we need a 'safemask'? we always want to encode the same. */ static int quoted_encode(const unsigned char *in, int len, unsigned char *out, unsigned short safemask) { @@ -797,25 +798,25 @@ quoted_encode(const unsigned char *in, int len, unsigned char *out, unsigned sho unsigned char c; inptr = in; - inend = in+len; + inend = in + len; outptr = out; - while (inptr<inend) { + while (inptr < inend) { c = *inptr++; - /*if (is_qpsafe(c) && !(c=='_' || c=='?')) {*/ - if (camel_mime_special_table[c] & safemask) { - if (c==' ') - c='_'; - *outptr++=c; + if ((is_qpsafe (c) || c == ' ') && !(c == '_' || c == '?')) { + /*if (camel_mime_special_table[c] & safemask) {*/ + if (c == ' ') + c = '_'; + *outptr++ = c; } else { *outptr++ = '='; - *outptr++ = tohex[(c>>4) & 0xf]; + *outptr++ = tohex[(c >> 4) & 0xf]; *outptr++ = tohex[c & 0xf]; } } - printf("encoding '%.*s' = '%.*s'\n", len, in, outptr-out, out); + d(printf("encoding '%.*s' = '%.*s'\n", len, in, outptr-out, out)); - return outptr-out; + return (outptr - out); } @@ -1177,9 +1178,9 @@ header_encode_string(const unsigned char *in) } start = inptr; encoding = 0; - } else if (c>127 && c < 256) { + } else if (c > 127 && c < 256) { encoding = MAX(encoding, 1); - } else if (c >=256) { + } else if (c >= 256) { encoding = MAX(encoding, 2); } } |