cleanup: leverage yield from in more places
Change-Id: I4f9cb27d89241d3738486764817b51981444a903
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/390274
Reviewed-by: Aravind Vasudevan <aravindvasudev@google.com>
Commit-Queue: Mike Frysinger <vapier@google.com>
Tested-by: Mike Frysinger <vapier@google.com>
diff --git a/command.py b/command.py
index a6189ed..8c31c18 100644
--- a/command.py
+++ b/command.py
@@ -476,8 +476,7 @@
top = self.manifest
yield top
if not opt.this_manifest_only:
- for child in top.all_children:
- yield child
+ yield from top.all_children
class InteractiveCommand(Command):
diff --git a/manifest_xml.py b/manifest_xml.py
index d2217bb..0392517 100644
--- a/manifest_xml.py
+++ b/manifest_xml.py
@@ -857,8 +857,7 @@
self._Load()
outer = self._outer_client
yield outer
- for tree in outer.all_children:
- yield tree
+ yield from outer.all_children
@property
def all_children(self):
@@ -867,8 +866,7 @@
for child in self._submanifests.values():
if child.repo_client:
yield child.repo_client
- for tree in child.repo_client.all_children:
- yield tree
+ yield from child.repo_client.all_children
@property
def path_prefix(self):
diff --git a/platform_utils.py b/platform_utils.py
index 2c48e62..d720a07 100644
--- a/platform_utils.py
+++ b/platform_utils.py
@@ -193,10 +193,9 @@
for name in dirs:
new_path = os.path.join(top, name)
if followlinks or not islink(new_path):
- for x in _walk_windows_impl(
+ yield from _walk_windows_impl(
new_path, topdown, onerror, followlinks
- ):
- yield x
+ )
if not topdown:
yield top, dirs, nondirs
