diff options
| author | Charles Harris <charlesr.harris@gmail.com> | 2013-04-06 19:09:17 -0600 |
|---|---|---|
| committer | Charles Harris <charlesr.harris@gmail.com> | 2013-04-06 19:09:17 -0600 |
| commit | aab46a78cefe9fbd46104496ffad17667784a3f5 (patch) | |
| tree | 1385a8433047a5cb92f3dcf3b2af12ca02b0025d /tools/c_coverage/c_coverage_report.py | |
| parent | 3c70e20a5f1aed4098ba66d21e6a1f60edc6fddd (diff) | |
| download | numpy-aab46a78cefe9fbd46104496ffad17667784a3f5.tar.gz | |
2to3: apply `dict` fixer.
In Python3 `dict.items()`, `dict.keys()`, and `dict.values()` are
iterators. This causes problems when a list is needed so the 2to3 fixer
explicitly constructs a list when is finds on of those functions.
However, that is usually not necessary, so a lot of the work here has
been cleaning up those places where the fix is not needed. The big
exception to that is the `numpy/f2py/crackfortran.py` file. The code
there makes extensive use of loops that modify the contents of the
dictionary being looped through, which raises an error. That together
with the obscurity of the code in that file made it safest to let the
`dict` fixer do its worst.
Closes #3050.
Diffstat (limited to 'tools/c_coverage/c_coverage_report.py')
| -rwxr-xr-x | tools/c_coverage/c_coverage_report.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/c_coverage/c_coverage_report.py b/tools/c_coverage/c_coverage_report.py index 9e29a7566..d1084b75e 100755 --- a/tools/c_coverage/c_coverage_report.py +++ b/tools/c_coverage/c_coverage_report.py @@ -110,7 +110,7 @@ class SourceFiles: fd = open(os.path.join(root, 'index.html'), 'w') fd.write("<html>") - paths = self.files.keys() + paths = list(self.files.keys()) paths.sort() for path in paths: fd.write('<p><a href="%s.html">%s</a></p>' % |