Mercurial > hg
changeset 17826:46e1a4e24225
obsolete: rename `anysuccessors` into `allsuccessors`
The "any" prefix looks like it returned a boolean. `allsuccessors` is more
accurate.
| author | Pierre-Yves David <pierre-yves.david@ens-lyon.org> |
|---|---|
| date | Fri, 19 Oct 2012 00:30:11 +0200 |
| parents | 3cc06457f15e |
| children | 612db9d7e76a |
| files | mercurial/bookmarks.py mercurial/discovery.py mercurial/obsolete.py |
| diffstat | 3 files changed, 3 insertions(+), 3 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/bookmarks.py Fri Oct 19 00:28:13 2012 +0200 +++ b/mercurial/bookmarks.py Fri Oct 19 00:30:11 2012 +0200 @@ -266,7 +266,7 @@ for c in validdests: if c.phase() > phases.public: # obsolescence marker does not apply to public changeset - succs.update(obsolete.anysuccessors(repo.obsstore, + succs.update(obsolete.allsuccessors(repo.obsstore, c.node())) validdests = set(repo.set('%ln::', succs)) validdests.remove(old)
--- a/mercurial/discovery.py Fri Oct 19 00:28:13 2012 +0200 +++ b/mercurial/discovery.py Fri Oct 19 00:30:11 2012 +0200 @@ -298,7 +298,7 @@ if nh in repo and repo[nh].phase() <= phases.public: newhs.add(nh) else: - for suc in obsolete.anysuccessors(repo.obsstore, nh): + for suc in obsolete.allsuccessors(repo.obsstore, nh): if suc != nh and suc in allfuturecommon: discardedheads.add(nh) break
--- a/mercurial/obsolete.py Fri Oct 19 00:28:13 2012 +0200 +++ b/mercurial/obsolete.py Fri Oct 19 00:30:11 2012 +0200 @@ -351,7 +351,7 @@ for data in ctx._repo.obsstore.successors.get(ctx.node(), ()): yield marker(ctx._repo, data) -def anysuccessors(obsstore, node): +def allsuccessors(obsstore, node): """Yield every successor of <node> This is a linear yield unsuited to detecting split changesets."""