inotify: inotify.server.walk*() remove unnecessary var
Remove hginside var and the test it relates to:
not( top or not hginside ) == (not top) and hginside, so the only case
when nothing will be yielded is when hginside is True and top is False.
Because of the returns placed upstream, this case will not happen anymore.
We can then safely remove hginside and the (if)s
--- a/hgext/inotify/server.py Mon May 04 17:17:03 2009 +0900
+++ b/hgext/inotify/server.py Mon May 04 17:58:26 2009 +0900
@@ -35,12 +35,10 @@
def walkit(dirname, top):
fullpath = rootslash + dirname
- hginside = False
try:
for name, kind in osutil.listdir(fullpath):
if kind == stat.S_IFDIR:
if name == '.hg':
- hginside = True
if not top:
return
else:
@@ -52,8 +50,7 @@
except OSError, err:
if err.errno not in walk_ignored_errors:
raise
- if top or not hginside:
- yield fullpath
+ yield fullpath
return walkit('', True)
@@ -66,14 +63,12 @@
def walkit(root, reporoot):
files, dirs = [], []
- hginside = False
try:
fullpath = rootslash + root
for name, kind in osutil.listdir(fullpath):
if kind == stat.S_IFDIR:
if name == '.hg':
- hginside = True
if reporoot:
continue
else:
@@ -81,8 +76,7 @@
dirs.append(name)
elif kind in (stat.S_IFREG, stat.S_IFLNK):
files.append((name, kind))
- if reporoot or not hginside:
- yield fullpath, dirs, files
+ yield fullpath, dirs, files
for subdir in dirs:
path = join(root, subdir)