mirror of
https://github.com/python/cpython.git
synced 2025-10-10 00:43:41 +00:00
gh-102498 Clean up unused variables and imports in the email module (#102482)
* Clean up unused variables and imports in the email module * Remove extra newline char * Remove superflous dict+unpacking syntax * Remove unused 'msg' var * Clean up unused variables and imports in the email module * Remove extra newline char * Remove superflous dict+unpacking syntax * Remove unused 'msg' var --------- Co-authored-by: Barry Warsaw <barry@python.org>
This commit is contained in:
parent
58b6be3791
commit
04ea04807d
5 changed files with 6 additions and 10 deletions
|
@ -1987,7 +1987,7 @@ def get_address_list(value):
|
||||||
try:
|
try:
|
||||||
token, value = get_address(value)
|
token, value = get_address(value)
|
||||||
address_list.append(token)
|
address_list.append(token)
|
||||||
except errors.HeaderParseError as err:
|
except errors.HeaderParseError:
|
||||||
leader = None
|
leader = None
|
||||||
if value[0] in CFWS_LEADER:
|
if value[0] in CFWS_LEADER:
|
||||||
leader, value = get_cfws(value)
|
leader, value = get_cfws(value)
|
||||||
|
@ -2096,7 +2096,7 @@ def get_msg_id(value):
|
||||||
except errors.HeaderParseError:
|
except errors.HeaderParseError:
|
||||||
try:
|
try:
|
||||||
token, value = get_no_fold_literal(value)
|
token, value = get_no_fold_literal(value)
|
||||||
except errors.HeaderParseError as e:
|
except errors.HeaderParseError:
|
||||||
try:
|
try:
|
||||||
token, value = get_domain(value)
|
token, value = get_domain(value)
|
||||||
msg_id.defects.append(errors.ObsoleteHeaderDefect(
|
msg_id.defects.append(errors.ObsoleteHeaderDefect(
|
||||||
|
@ -2443,7 +2443,6 @@ def get_parameter(value):
|
||||||
raise errors.HeaderParseError("Parameter not followed by '='")
|
raise errors.HeaderParseError("Parameter not followed by '='")
|
||||||
param.append(ValueTerminal('=', 'parameter-separator'))
|
param.append(ValueTerminal('=', 'parameter-separator'))
|
||||||
value = value[1:]
|
value = value[1:]
|
||||||
leader = None
|
|
||||||
if value and value[0] in CFWS_LEADER:
|
if value and value[0] in CFWS_LEADER:
|
||||||
token, value = get_cfws(value)
|
token, value = get_cfws(value)
|
||||||
param.append(token)
|
param.append(token)
|
||||||
|
@ -2568,7 +2567,7 @@ def parse_mime_parameters(value):
|
||||||
try:
|
try:
|
||||||
token, value = get_parameter(value)
|
token, value = get_parameter(value)
|
||||||
mime_parameters.append(token)
|
mime_parameters.append(token)
|
||||||
except errors.HeaderParseError as err:
|
except errors.HeaderParseError:
|
||||||
leader = None
|
leader = None
|
||||||
if value[0] in CFWS_LEADER:
|
if value[0] in CFWS_LEADER:
|
||||||
leader, value = get_cfws(value)
|
leader, value = get_cfws(value)
|
||||||
|
@ -2626,7 +2625,6 @@ def parse_content_type_header(value):
|
||||||
don't do that.
|
don't do that.
|
||||||
"""
|
"""
|
||||||
ctype = ContentType()
|
ctype = ContentType()
|
||||||
recover = False
|
|
||||||
if not value:
|
if not value:
|
||||||
ctype.defects.append(errors.HeaderMissingRequiredValue(
|
ctype.defects.append(errors.HeaderMissingRequiredValue(
|
||||||
"Missing content type specification"))
|
"Missing content type specification"))
|
||||||
|
|
|
@ -341,7 +341,6 @@ class Charset:
|
||||||
if not lines and not current_line:
|
if not lines and not current_line:
|
||||||
lines.append(None)
|
lines.append(None)
|
||||||
else:
|
else:
|
||||||
separator = (' ' if lines else '')
|
|
||||||
joined_line = EMPTYSTRING.join(current_line)
|
joined_line = EMPTYSTRING.join(current_line)
|
||||||
header_bytes = _encode(joined_line, codec)
|
header_bytes = _encode(joined_line, codec)
|
||||||
lines.append(encoder(header_bytes))
|
lines.append(encoder(header_bytes))
|
||||||
|
|
|
@ -264,7 +264,7 @@ class FeedParser:
|
||||||
yield NeedMoreData
|
yield NeedMoreData
|
||||||
continue
|
continue
|
||||||
break
|
break
|
||||||
msg = self._pop_message()
|
self._pop_message()
|
||||||
# We need to pop the EOF matcher in order to tell if we're at
|
# We need to pop the EOF matcher in order to tell if we're at
|
||||||
# the end of the current file, not the end of the last block
|
# the end of the current file, not the end of the last block
|
||||||
# of message headers.
|
# of message headers.
|
||||||
|
|
|
@ -14,7 +14,7 @@ from io import BytesIO, StringIO
|
||||||
# Intrapackage imports
|
# Intrapackage imports
|
||||||
from email import utils
|
from email import utils
|
||||||
from email import errors
|
from email import errors
|
||||||
from email._policybase import Policy, compat32
|
from email._policybase import compat32
|
||||||
from email import charset as _charset
|
from email import charset as _charset
|
||||||
from email._encoded_words import decode_b
|
from email._encoded_words import decode_b
|
||||||
Charset = _charset.Charset
|
Charset = _charset.Charset
|
||||||
|
|
|
@ -6,7 +6,6 @@
|
||||||
|
|
||||||
__all__ = ['MIMEText']
|
__all__ = ['MIMEText']
|
||||||
|
|
||||||
from email.charset import Charset
|
|
||||||
from email.mime.nonmultipart import MIMENonMultipart
|
from email.mime.nonmultipart import MIMENonMultipart
|
||||||
|
|
||||||
|
|
||||||
|
@ -36,6 +35,6 @@ class MIMEText(MIMENonMultipart):
|
||||||
_charset = 'utf-8'
|
_charset = 'utf-8'
|
||||||
|
|
||||||
MIMENonMultipart.__init__(self, 'text', _subtype, policy=policy,
|
MIMENonMultipart.__init__(self, 'text', _subtype, policy=policy,
|
||||||
**{'charset': str(_charset)})
|
charset=str(_charset))
|
||||||
|
|
||||||
self.set_payload(_text, _charset)
|
self.set_payload(_text, _charset)
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue