[Numpy-discussion] Import patch
Paul Anton Letnes
paul.anton.letnes at gmail.com
Sun Apr 3 06:35:08 EDT 2011
Hi.
When looking at the loadtxt/savetxt tickets, I noticed that the 're' module is imported in an odd place. I therefore suggest that this import is moved to the top of the file, in order to gather these as much as possible. I find the code easier to read then. After all, there is no 'try / catch' or similar to check if the module exists. See patch below. I do not believe any tests or tickets are needed - correct me if I am wrong.
Cheers,
Paul.
--- a/numpy/lib/npyio.py Sat Apr 02 20:19:55 2011 -0600
+++ b/numpy/lib/npyio.py Sun Apr 03 12:30:02 2011 +0200
@@ -6,6 +6,7 @@
import format
import sys
import os
+import re
import sys
import itertools
import warnings
@@ -956,7 +957,6 @@
if own_fh:
fh.close()
-import re
def fromregex(file, regexp, dtype):
"""
Construct an array from a text file, using regular expression parsing.
More information about the NumPy-Discussion
mailing list