diff options
author | Johannes Gijsbers <jlg@dds.nl> | 2005-01-08 13:13:19 (GMT) |
---|---|---|
committer | Johannes Gijsbers <jlg@dds.nl> | 2005-01-08 13:13:19 (GMT) |
commit | 836f5433f7ee79f208186ca0d10594b22cf7e05b (patch) | |
tree | d5909575b732a17a75dc5e0e76180d61a5d037c6 /Demo | |
parent | e4172eadf3fb9c1de591305ad4ca4ce3e252abd3 (diff) | |
download | cpython-836f5433f7ee79f208186ca0d10594b22cf7e05b.zip cpython-836f5433f7ee79f208186ca0d10594b22cf7e05b.tar.gz cpython-836f5433f7ee79f208186ca0d10594b22cf7e05b.tar.bz2 |
Patch #943206:
`glob.glob()` currently calls itself recursively to build a list of matches of
the dirname part of the pattern and then filters by the basename part. This is
effectively BFS. ``glob.glob('*/*/*/*/*/foo')`` will build a huge list of all
directories 5 levels deep even if only a handful of them contain a ``foo``
entry. A generator-based recusion would never have to store these list at once
by implementing DFS. This patch converts the `glob` function to an `iglob`
recursive generator . `glob()` now just returns ``list(iglob(pattern))``.
I also cleaned up the code a bit (reduced duplicate `has_magic()` checks and
created a second `glob0` helper func so that the main loop need not be
duplicated).
Thanks to Cherniavsky Beni for the patch!
Diffstat (limited to 'Demo')
0 files changed, 0 insertions, 0 deletions