diff options
| author | Recursing <buonanno.lorenzo@gmail.com> | 2018-12-23 03:48:14 (GMT) |
|---|---|---|
| committer | Raymond Hettinger <rhettinger@users.noreply.github.com> | 2018-12-23 03:48:14 (GMT) |
| commit | 3ce3dea60646d8a5a1c952469a2eb65f937875b3 (patch) | |
| tree | 145d309073b9d7db0e48a2f0e13822c4a7374492 /Lib/os.py | |
| parent | fc8284e22074af8154e9865c8391b955f13a308b (diff) | |
| download | cpython-3ce3dea60646d8a5a1c952469a2eb65f937875b3.zip cpython-3ce3dea60646d8a5a1c952469a2eb65f937875b3.tar.gz cpython-3ce3dea60646d8a5a1c952469a2eb65f937875b3.tar.bz2 | |
Use generator instead of list in code examples (GH-11203)
There is no need to create a list for `sum`
Also, becomes consistent with the first example in Doc/library/os.rst
Diffstat (limited to 'Lib/os.py')
| -rw-r--r-- | Lib/os.py | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -327,7 +327,7 @@ def walk(top, topdown=True, onerror=None, followlinks=False): from os.path import join, getsize for root, dirs, files in os.walk('python/Lib/email'): print(root, "consumes", end="") - print(sum([getsize(join(root, name)) for name in files]), end="") + print(sum(getsize(join(root, name)) for name in files), end="") print("bytes in", len(files), "non-directory files") if 'CVS' in dirs: dirs.remove('CVS') # don't visit CVS directories @@ -446,7 +446,7 @@ if {open, stat} <= supports_dir_fd and {scandir, stat} <= supports_fd: import os for root, dirs, files, rootfd in os.fwalk('python/Lib/email'): print(root, "consumes", end="") - print(sum([os.stat(name, dir_fd=rootfd).st_size for name in files]), + print(sum(os.stat(name, dir_fd=rootfd).st_size for name in files), end="") print("bytes in", len(files), "non-directory files") if 'CVS' in dirs: |