Commit 3ce3dea6 authored by Recursing's avatar Recursing Committed by Raymond Hettinger

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
parent fc8284e2
...@@ -327,7 +327,7 @@ def walk(top, topdown=True, onerror=None, followlinks=False): ...@@ -327,7 +327,7 @@ def walk(top, topdown=True, onerror=None, followlinks=False):
from os.path import join, getsize from os.path import join, getsize
for root, dirs, files in os.walk('python/Lib/email'): for root, dirs, files in os.walk('python/Lib/email'):
print(root, "consumes", end="") 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") print("bytes in", len(files), "non-directory files")
if 'CVS' in dirs: if 'CVS' in dirs:
dirs.remove('CVS') # don't visit CVS directories dirs.remove('CVS') # don't visit CVS directories
...@@ -446,7 +446,7 @@ if {open, stat} <= supports_dir_fd and {scandir, stat} <= supports_fd: ...@@ -446,7 +446,7 @@ if {open, stat} <= supports_dir_fd and {scandir, stat} <= supports_fd:
import os import os
for root, dirs, files, rootfd in os.fwalk('python/Lib/email'): for root, dirs, files, rootfd in os.fwalk('python/Lib/email'):
print(root, "consumes", end="") 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="") end="")
print("bytes in", len(files), "non-directory files") print("bytes in", len(files), "non-directory files")
if 'CVS' in dirs: if 'CVS' in dirs:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment