From patchwork Mon Sep 10 13:15:19 2018
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: ancestor: remove extra generator from lazyancestors.__iter__()
From: Yuya Nishihara <yuya@tcha.org>
X-Patchwork-Id: 34454
Message-Id: <4eee6e03686df30cacc8.1536585319@mimosa>
To: mercurial-devel@mercurial-scm.org
Date: Mon, 10 Sep 2018 22:15:19 +0900

# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 1536580638 -32400
#      Mon Sep 10 20:57:18 2018 +0900
# Node ID 4eee6e03686df30cacc8d78f9bd37d4628e478d6
# Parent  43d92d68ac889eed34e6223bdc0298bc4f056592
ancestor: remove extra generator from lazyancestors.__iter__()

diff --git a/mercurial/ancestor.py b/mercurial/ancestor.py
--- a/mercurial/ancestor.py
+++ b/mercurial/ancestor.py
@@ -335,9 +335,8 @@ class lazyancestors(object):
 
         If inclusive is True, the source revisions are also yielded. The
         reverse revision number order is still enforced."""
-        for rev in _lazyancestorsiter(self._parentrevs, self._initrevs,
-                                      self._stoprev, self._inclusive):
-            yield rev
+        return _lazyancestorsiter(self._parentrevs, self._initrevs,
+                                  self._stoprev, self._inclusive)
 
     def __contains__(self, target):
         """Test whether target is an ancestor of self._initrevs."""