Mercurial > public > mercurial-scm > hg-stable
diff mercurial/hgweb/hgwebdir_mod.py @ 52669:e627cc25b6f3
pyupgrade: rewrite `yield` statements in a loop to `yield from`
This is the `legacy` fixer in `pyupgrade`, with the `yield` statement yielding
loop commented back in. This seems to help pytype in some cases, and hurt it in
others. But that can be manually fixed later.
Note that it's possibly buggy in that it aggressively changed `import-checker.py`
to `yield from 'fcntl', 'grp', 'pwd', 'select', 'termios': # Unix only`, which
is invalid syntax. Possibly it needed help from the token fixer that I've
disabled locally (because that wants to make a bunch of unrelated changes).
Just change those few places to yield from a list, to avoid having to constantly
revert that.
| author | Matt Harbison <matt_harbison@yahoo.com> |
|---|---|
| date | Sun, 05 Jan 2025 22:26:16 -0500 |
| parents | 24ee91ba9aa8 |
| children |
line wrap: on
line diff
--- a/mercurial/hgweb/hgwebdir_mod.py Sun Jan 05 22:23:31 2025 -0500 +++ b/mercurial/hgweb/hgwebdir_mod.py Sun Jan 05 22:26:16 2025 -0500 @@ -382,8 +382,7 @@ profile = self.ui.configbool(b'profiling', b'enabled') with profiling.profile(self.ui, enabled=profile): try: - for r in self._runwsgi(req, res): - yield r + yield from self._runwsgi(req, res) finally: # There are known cycles in localrepository that prevent # those objects (and tons of held references) from being @@ -452,8 +451,7 @@ def _virtualdirs(): # Check the full virtual path, and each parent yield virtual - for p in pathutil.finddirs(virtual): - yield p + yield from pathutil.finddirs(virtual) for virtualrepo in _virtualdirs(): real = repos.get(virtualrepo)