[Numpy-discussion] Import patch

[Paul Anton Letnes]

On 4. apr. 2011, at 15.34, Charles R Harris wrote:

> 
> 
> On Sun, Apr 3, 2011 at 4:35 AM, Paul Anton Letnes <paul.anton.letnes at gmail.com> wrote:
> 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.
> 
> If you want to see a lot of other small things for cleanups, run pyflakes or pylint on the files in numpy/lib
> 
> Chuck
> _______________________________________________
> NumPy-Discussion mailing list
> NumPy-Discussion at scipy.org
> http://mail.scipy.org/mailman/listinfo/numpy-discussion

Are you suggesting that I should do this and submit one or more patches?

By the way - what is a suggested form of submitting patches? Using e-mail seems a bit clumsy to me.

Paul.