This is an archive of the discontinued Mercurial Phabricator instance.

py3: cast character set to bytes
ClosedPublic

Authored by indygreg on Feb 11 2018, 10:45 PM.

Download Raw Diff

Details

Reviewers

pulkit
durin42

Group Reviewers

hg-reviewers

Commits

rHG6ea7f1c10c81: py3: cast character set to bytes

Summary

The returned value from this function is eventually fed into
email.charset.Charset.init. However, the standard library
appears to handle both string types in Python 2 and 3. So there
shouldn't be any problem returning a bytes here.

Diff Detail

Repository

rHG Mercurial

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

indygreg created this revision.Feb 11 2018, 10:45 PM

Herald added a reviewer: hg-reviewers. · View Herald TranscriptFeb 11 2018, 10:45 PM

Herald added a subscriber: mercurial-devel. · View Herald Transcript

indygreg added a child revision: D2168: mail: import email.utils not email.Utils.Feb 11 2018, 10:45 PM

pulkit accepted this revision.Feb 12 2018, 2:43 AM

durin42 accepted this revision.Feb 12 2018, 9:31 PM

This revision is now accepted and ready to land.Feb 12 2018, 9:31 PM

Closed by commit rHG6ea7f1c10c81: py3: cast character set to bytes (authored by indygreg). · Explain WhyFeb 12 2018, 9:47 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			mercurial/mail.py (2 lines)

Status	Author	Revision
Closed	indygreg	D2151 py3: check for bytes instead of str in hg.share()
Closed	indygreg	D2150 py3: port metaedit extension to Python 3
Closed	indygreg	D2149 py3: pass system string to email.message.Message.set_type()
Closed	indygreg	D2148 py3: open temporary file in binary mode
Closed	indygreg	D2147 py3: use email parser that operates on bytes
Closed	indygreg	D2176 py3: convert traceback representation to bytes when logging
Closed	indygreg	D2175 py3: use b'' in mockblackbox.py
Closed	indygreg	D2174 py3: convert context to bytes instead of str
Closed	indygreg	D2173 py3: add missing b'' literal to sshprotoext.py
Closed	indygreg	D2172 py3: convert FrameInfo members to bytes
Closed	indygreg	D2171 ui: use named attributes on FrameInfo instance
Closed	indygreg	D2170 py3: make dummyssh compatible with Python 3
Closed	indygreg	D2169 py3: explicitly cast bool to bytes
Closed	indygreg	D2168 mail: import email.utils not email.Utils
Closed	indygreg	D2167 py3: cast character set to bytes
Closed	indygreg	D2166 py3: cast decode() argument to system string
Closed	indygreg	D2165 githelp: cast commands to bytes
Closed	indygreg	D2164 py3: avoid changing dictionary during iteration
Closed	indygreg	D2163 py3: use raw string for key in **kwargs
Closed	indygreg	D2162 py3: preserve chunks as an iterable of bytes
Closed	indygreg	D2161 py3: use b'' for changegroup version literals
Closed	indygreg	D2160 py3: use b'' in inline extension
Closed	indygreg	D2159 py3: use hex(hasher.digest())
Needs Revision	indygreg	D2158 py3: use raw string for open() mode
Closed	indygreg	D2157 py3: use string for "close" value in commit extras
Closed	indygreg	D2156 py3: catch TypeError during template operations
Closed	indygreg	D2155 py3: use bytes literals for test extension
Closed	indygreg	D2154 py3: use system strings when calling __import__
Closed	indygreg	D2153 py3: open patches.queue in binary mode
Closed	indygreg	D2152 py3: compare against bytes instead of str
Closed	indygreg	D2146 py3: port string formatting
Closed	indygreg	D2145 py3: port ext-phase-report.py extension
Closed	indygreg	D2144 py3: more robustly cast UUID to bytes
Closed	indygreg	D2143 py3: use bytes() to cast context instances
Closed	indygreg	D2142 py3: convert __doc__ to bytes
Closed	indygreg	D2141 py3: cast repr() result to bytes
Closed	indygreg	D2140 py3: fix file i/o in test-status.t
Closed	indygreg	D2139 py3: pass system string to socket.getservbyname
Closed	indygreg	D2138 run-tests: report tests that exception occurred in
Closed	indygreg	D2137 py3: use raw string for file open mode
Closed	indygreg	D2136 py3: convert content-type to bytes

Diff 5590

mercurial/mail.py

	'but no smtp host configured'))			'but no smtp host configured'))
	else:			else:
	if not util.findexe(method):			if not util.findexe(method):
	raise error.Abort(_('%r specified as email transport, '			raise error.Abort(_('%r specified as email transport, '
	'but not in PATH') % method)			'but not in PATH') % method)

	def codec2iana(cs):			def codec2iana(cs):
	''''''			''''''
	cs = email.charset.Charset(cs).input_charset.lower()			cs = pycompat.sysbytes(email.charset.Charset(cs).input_charset.lower())

	# "latin1" normalizes to "iso8859-1", standard calls for "iso-8859-1"			# "latin1" normalizes to "iso8859-1", standard calls for "iso-8859-1"
	if cs.startswith("iso") and not cs.startswith("iso-"):			if cs.startswith("iso") and not cs.startswith("iso-"):
	return "iso-" + cs[3:]			return "iso-" + cs[3:]
	return cs			return cs

	def mimetextpatch(s, subtype='plain', display=False):			def mimetextpatch(s, subtype='plain', display=False):
	'''Return MIME message suitable for a patch.			'''Return MIME message suitable for a patch.

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	5468		Feb 11 2018, 10:45 PM	★	★
Diff 2	5590	rHG6ea7f1c10c8160da6e31e30c9a024feab4980e15	Feb 11 2018, 9:50 PM	★	★