Éric Araujo <mer...@netwok.org> added the comment:

Excellent!  I will apply this.

> I looked through the rest of database.py and did not see any other
> generators that appeared to erroneously hold a file handle open.
Me neither.  Thanks for checking.

> In addition there is one generator function (list_distinfo_files)
> which feeds off of the generator I have changed but I didn't see a
> need to alter them as they work fine as generators.
I think using a generator can save some memory, if the underlying list is huge, 
but the important thing here is that all functions with a similar name (list_*) 
behave in a consistent way, i.e. returning lists or generators but not a mix.

----------
priority: normal -> high

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue12504>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to