diff options
author | Dylan Baker <dylan@pnwbakers.com> | 2021-08-31 10:08:01 -0700 |
---|---|---|
committer | Eli Schwartz <eschwartz93@gmail.com> | 2021-08-31 16:28:54 -0400 |
commit | b60bd0e299460c436acba43de27ac52afb11026b (patch) | |
tree | d1e66beb2c83625bfc86acbac7ce419980fd5013 /mesonbuild/backend/ninjabackend.py | |
parent | 4d7031437c7a81b52c776d4ae1e32741bdb851ca (diff) | |
download | meson-b60bd0e299460c436acba43de27ac52afb11026b.zip meson-b60bd0e299460c436acba43de27ac52afb11026b.tar.gz meson-b60bd0e299460c436acba43de27ac52afb11026b.tar.bz2 |
pyllint: enable consider-user-enumerate
This caught a couple of cases of us doing:
```python
for i in range(len(x)):
v = x[i]
```
which are places to use enumerate instead.
It also caught a couple of cases of:
```python
assert len(x) == len(y)
for i in range(len(x)):
xv = x[i]
yv = y[i]
```
Which should instead be using zip()
```python
for xv, yv in zip(x, y):
...
```
Diffstat (limited to 'mesonbuild/backend/ninjabackend.py')
-rw-r--r-- | mesonbuild/backend/ninjabackend.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mesonbuild/backend/ninjabackend.py b/mesonbuild/backend/ninjabackend.py index ca9ab9a..204510e 100644 --- a/mesonbuild/backend/ninjabackend.py +++ b/mesonbuild/backend/ninjabackend.py @@ -2193,8 +2193,7 @@ https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47485''')) infilelist = genlist.get_inputs() outfilelist = genlist.get_outputs() extra_dependencies = self.get_custom_target_depend_files(genlist) - for i in range(len(infilelist)): - curfile = infilelist[i] + for i, curfile in enumerate(infilelist): if len(generator.outputs) == 1: sole_output = os.path.join(self.get_target_private_dir(target), outfilelist[i]) else: |