This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	durin42	D5210 statprof: clean up unicode/bytes a little
Closed	durin42	D5209 statprof: add a couple of asserts to avoid storing unicodes
Closed	durin42	D5208 tests: add some helpful `\|\| cat` bits to test-profile.t
Closed	durin42	D5207 tests: skip wireproto clientreactor tests on Python 3.6.0-3.6.3 inclusive
Closed	durin42	D5206 contrib: tweak import-checker to always use bytes for module names
Closed	durin42	D5205 contrib: fix import-checker to not b'' module names on Python 3
Closed	durin42	D5204 tests: add a critical flush() to run-tests.py to make output stable on py3

Diff 12382

	for name, starts, ends, code in testparseutil.pyembedded(f, src, errors):			for name, starts, ends, code in testparseutil.pyembedded(f, src, errors):
	if not name:			if not name:
	# use 'doctest.py', in order to make already existing			# use 'doctest.py', in order to make already existing
	# doctest above pass instantly			# doctest above pass instantly
	name = 'doctest.py'			name = 'doctest.py'
	# "starts" is "line number" (1-origin), but embedded() is			# "starts" is "line number" (1-origin), but embedded() is
	# expected to return "line offset" (0-origin). Therefore, this			# expected to return "line offset" (0-origin). Therefore, this
	# yields "starts - 1".			# yields "starts - 1".
				if not isinstance(modname, str):
				modname = modname.decode('utf8')
	yield code, "%s[%d]" % (modname, starts), name, starts - 1			yield code, "%s[%d]" % (modname, starts), name, starts - 1

	def sources(f, modname):			def sources(f, modname):
	"""Yields possibly multiple sources from a filepath			"""Yields possibly multiple sources from a filepath

	input: filepath, modulename			input: filepath, modulename
	yields: script(string), modulename, filepath, linenumber			yields: script(string), modulename, filepath, linenumber