0ct0pu5/wesnoth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use for i in iterable instead of for i in range(len(iterable)). Have a hard time testing it well, but it seems to not break anything.

Browse source
This commit is contained in:
Thibault Févry 2013-08-08 23:35:11 +02:00
parent 5a5287b1f7
commit e3fbfe5225
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
data/tools/wesnoth/wmltools.py
View file

@ -89,15 +89,15 @@ class Forest:
# Always look at _main.cfg first
subtree.sort(lambda x, y: cmp(x, y) - 2*int(x.endswith("_main.cfg")) + 2*int(y.endswith("_main.cfg")))
self.forest.append(subtree)
for i in range(len(self.forest)):
for i in self.forest:
# Ignore version-control subdirectories and Emacs tempfiles
for dirkind in vc_directories + l10n_directories:
self.forest[i] = filter(lambda x: dirkind not in x, self.forest[i])
self.forest[i] = filter(lambda x: '.#' not in x, self.forest[i])
self.forest[i] = filter(lambda x: not os.path.isdir(x), self.forest[i])
i = filter(lambda x: dirkind not in x, i)
i = filter(lambda x: '.#' not in x, i)
i = filter(lambda x: not os.path.isdir(x), i)
if exclude:
self.forest[i] = filter(lambda x: not re.search(exclude, x), self.forest[i])
self.forest[i] = filter(lambda x: not x.endswith("-bak"), self.forest[i])
i = filter(lambda x: not re.search(exclude, x), i)
i = filter(lambda x: not x.endswith("-bak"), i)
# Compute cliques (will be used later for visibility checks)
self.clique = {}
counter = 0