From: R. David Murray Date: Wed, 5 May 2010 17:31:03 +0000 (+0000) Subject: Issue #7472: remove unused code from email.encoders.encode_7or8bit. X-Git-Tag: v2.7b2~78 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7d93221a5c159ce506b2a557777d1a1751c7f743;p=python Issue #7472: remove unused code from email.encoders.encode_7or8bit. Yukihiro Nakadaira noticed a typo in encode_7or8bit that was trying to special case iso-2022 codecs. It turns out that the code in question is never used, because whereas it was designed to trigger if the payload encoding was eight bit but its output encoding was 7 bit, in practice the payload is always converted to the 7bit encoding before encode_7or8bit is called. Patch by Shawat Anand. --- diff --git a/Lib/email/encoders.py b/Lib/email/encoders.py index c1a44aa88c..af45e62c33 100644 --- a/Lib/email/encoders.py +++ b/Lib/email/encoders.py @@ -72,13 +72,7 @@ def encode_7or8bit(msg): try: orig.encode('ascii') except UnicodeError: - # iso-2022-* is non-ASCII but still 7-bit - charset = msg.get_charset() - output_cset = charset and charset.output_charset - if output_cset and output_cset.lower().startswith('iso-2022-'): - msg['Content-Transfer-Encoding'] = '7bit' - else: - msg['Content-Transfer-Encoding'] = '8bit' + msg['Content-Transfer-Encoding'] = '8bit' else: msg['Content-Transfer-Encoding'] = '7bit' diff --git a/Lib/email/test/test_email.py b/Lib/email/test/test_email.py index 01d2a672a2..bf41be7570 100644 --- a/Lib/email/test/test_email.py +++ b/Lib/email/test/test_email.py @@ -564,6 +564,13 @@ class TestEncoders(unittest.TestCase): msg = MIMEText('hello \xf8 world', _charset='iso-8859-1') eq(msg['content-transfer-encoding'], 'quoted-printable') + def test_encode7or8bit(self): + # Make sure a charset whose input character set is 8bit but + # whose output character set is 7bit gets a transfer-encoding + # of 7bit. + eq = self.assertEqual + msg = email.MIMEText.MIMEText('\xca\xb8', _charset='euc-jp') + eq(msg['content-transfer-encoding'], '7bit') # Test long header wrapping