This is an archive of the discontinued Mercurial Phabricator instance.

mercurial/manifest.py
722–723	`test-check-pyflakes.t` didn't like this unused variable. I fixed it in flight. (So no action needed from you, and no need for others to send a fix for it.)

Revision Contents
Changeset List

			Path	Packages
M			mercurial/manifest.py (10 lines)

Commit	Parents	Author	Summary	Date
		spectral		Oct 2 2018, 4:38 PM

Status	Author	Revision
Closed	spectral	D4845 treemanifests: remove _loadalllazy in _diff()
Closed	spectral	D4844 treemanifests: skip extraneous check for item before calling _loadlazy
Closed	spectral	D4843 treemanifests: make _loadchildrensetlazy just call _loadlazy
Closed	spectral	D4842 treemanifests: make _loadlazy tolerate item not on _lazydirs


	def _loadchildrensetlazy(self, visit):			def _loadchildrensetlazy(self, visit):
	if not visit:			if not visit:
	return None			return None
	if visit == 'all' or visit == 'this':			if visit == 'all' or visit == 'this':
	self._loadalllazy()			self._loadalllazy()
	return None			return None

	todel = []			todel = []
				loadlazy = self._loadlazy
				martinvonzUnsubmitted Not Done `test-check-pyflakes.t` didn't like this unused variable. I fixed it in flight. (So no action needed from you, and no need for others to send a fix for it.) martinvonz: `test-check-pyflakes.t` didn't like this unused variable. I fixed it in flight. (So no action…
	for k in visit:			for k in visit:
	kslash = k + '/'			loadlazy(k + '/')
	ld = self._lazydirs.get(kslash)
	if ld:
	path, node, readsubtree = ld
	self._dirs[kslash] = readsubtree(path, node)
	todel.append(kslash)
	for kslash in todel:
	del self._lazydirs[kslash]
	return visit			return visit

	def __len__(self):			def __len__(self):
	self._load()			self._load()
	size = len(self._files)			size = len(self._files)
	self._loadalllazy()			self._loadalllazy()
	for m in self._dirs.values():			for m in self._dirs.values():
	size += m.__len__()			size += m.__len__()