This is an archive of the discontinued Mercurial Phabricator instance.

Differential D2371

walkrepos: don't reimplement any()
ClosedPublic

Authored by martinvonz on Feb 21 2018, 1:45 PM.

Details

Reviewers
durin42
Group Reviewers
hg-reviewers
Commits
rHGddd9474d2e08: walkrepos: don't reimplement any()

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

martinvonz created this revision.Feb 21 2018, 1:45 PM
durin42 accepted this revision.Feb 21 2018, 3:37 PM
This revision is now accepted and ready to land.Feb 21 2018, 3:37 PM
Closed by commit rHGddd9474d2e08: walkrepos: don't reimplement any() (authored by martinvonz). · Explain WhyFeb 21 2018, 3:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/scmutil.py (6 lines)

Diff 5951

mercurial/scmutil.py

'''yield every hg repository under path, always recursively. '''yield every hg repository under path, always recursively.
The recurse flag will only control recursion into repo working dirs''' The recurse flag will only control recursion into repo working dirs'''
def errhandler(err): def errhandler(err):
if err.filename == path: if err.filename == path:
raise err raise err
samestat = getattr(os.path, 'samestat', None) samestat = getattr(os.path, 'samestat', None)
if followsym and samestat is not None: if followsym and samestat is not None:
def adddir(dirlst, dirname): def adddir(dirlst, dirname):
match = False
dirstat = os.stat(dirname) dirstat = os.stat(dirname)
for lstdirstat in dirlst: match = any(samestat(dirstat, lstdirstat) for lstdirstat in dirlst)
if samestat(dirstat, lstdirstat):
match = True
break
if not match: if not match:
dirlst.append(dirstat) dirlst.append(dirstat)
return not match return not match
else: else:
followsym = False followsym = False
if (seen_dirs is None) and followsym: if (seen_dirs is None) and followsym:
seen_dirs = [] seen_dirs = []