diff options
| author | Charles Harris <charlesr.harris@gmail.com> | 2014-07-30 16:48:11 -0600 |
|---|---|---|
| committer | Julian Taylor <jtaylor.debian@googlemail.com> | 2014-07-31 21:21:13 +0200 |
| commit | dec6658cdc10a23ad0e733fb52a814306033d88c (patch) | |
| tree | dfa2a5d879f9ec10f75287a045070729cf415432 /numpy/lib/recfunctions.py | |
| parent | 0b5a6645ee110a8d4c6b5defd8c01791ee48bda1 (diff) | |
| download | numpy-dec6658cdc10a23ad0e733fb52a814306033d88c.tar.gz | |
MAINT: Fixes for problems in numpy/lib revealed by pyflakes.
Some of those problems look like potential coding errors. In those
cases a Fixme comment was made and the offending code, usually an
unused variable, was commented out.
Diffstat (limited to 'numpy/lib/recfunctions.py')
| -rw-r--r-- | numpy/lib/recfunctions.py | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/numpy/lib/recfunctions.py b/numpy/lib/recfunctions.py index ae4ee56c6..6ae8b8445 100644 --- a/numpy/lib/recfunctions.py +++ b/numpy/lib/recfunctions.py @@ -22,16 +22,13 @@ if sys.version_info[0] < 3: _check_fill_value = np.ma.core._check_fill_value -__all__ = ['append_fields', - 'drop_fields', - 'find_duplicates', - 'get_fieldstructure', - 'join_by', - 'merge_arrays', - 'rec_append_fields', 'rec_drop_fields', 'rec_join', - 'recursive_fill_fields', 'rename_fields', - 'stack_arrays', - ] + +__all__ = [ + 'append_fields', 'drop_fields', 'find_duplicates', + 'get_fieldstructure', 'join_by', 'merge_arrays', + 'rec_append_fields', 'rec_drop_fields', 'rec_join', + 'recursive_fill_fields', 'rename_fields', 'stack_arrays', + ] def recursive_fill_fields(input, output): @@ -896,7 +893,9 @@ def join_by(key, r1, r2, jointype='inner', r1postfix='1', r2postfix='2', # Make sure we work with ravelled arrays r1 = r1.ravel() r2 = r2.ravel() - (nb1, nb2) = (len(r1), len(r2)) + # Fixme: nb2 below is never used. Commenting out for pyflakes. + # (nb1, nb2) = (len(r1), len(r2)) + nb1 = len(r1) (r1names, r2names) = (r1.dtype.names, r2.dtype.names) # Check the names for collision |