This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	spectral	D4161 dirstate: add a comment on why we don't need to check if something is a dir/file
Closed	spectral	D4160 match: add missing "return set()", add FIXME to test to doc a bug
Closed	spectral	D4159 match: correct doc for _rootsdirsandparents after 5a7df82de142

Diff 10085

	def _roots(kindpats):			def _roots(kindpats):
	'''Returns root directories to match recursively from the given patterns.'''			'''Returns root directories to match recursively from the given patterns.'''
	roots, dirs = _patternrootsanddirs(kindpats)			roots, dirs = _patternrootsanddirs(kindpats)
	return roots			return roots

	def _rootsdirsandparents(kindpats):			def _rootsdirsandparents(kindpats):
	'''Returns roots and exact directories from patterns.			'''Returns roots and exact directories from patterns.

	roots are directories to match recursively, whereas exact directories should			`roots` are directories to match recursively, `dirs` should
	be matched non-recursively. The returned (roots, dirs) tuple will also			be matched non-recursively, and `parents` are the implicitly required
	include directories that need to be implicitly considered as either, such as			directories to walk to items in either roots or dirs.
	parent directories.
				Returns a tuple of (roots, dirs, parents).

	>>> _rootsdirsandparents(			>>> _rootsdirsandparents(
	... [(b'glob', b'g/h/*', b''), (b'glob', b'g/h', b''),			... [(b'glob', b'g/h/*', b''), (b'glob', b'g/h', b''),
	... (b'glob', b'g*', b'')])			... (b'glob', b'g*', b'')])
	(['g/h', 'g/h', '.'], [], ['g', '.'])			(['g/h', 'g/h', '.'], [], ['g', '.'])
	>>> _rootsdirsandparents(			>>> _rootsdirsandparents(
	... [(b'rootfilesin', b'g/h', b''), (b'rootfilesin', b'', b'')])			... [(b'rootfilesin', b'g/h', b''), (b'rootfilesin', b'', b'')])
	([], ['g/h', '.'], ['g', '.'])			([], ['g/h', '.'], ['g', '.'])