copies: rename some function to the new naming scheme
authorPierre-Yves David <pierre-yves.david@octobus.net>
Fri, 25 Sep 2020 14:33:05 +0200
changeset 45624 fb000408bca5
parent 45623 d31483377673
child 45625 c11099cc1de4
copies: rename some function to the new naming scheme We are about to introduce more function and more variants, it seems better to align everything on the new scheme, allowing `_` for clarify. We do this aligned in a dedicated changeset to make the next changesets simpler. Differential Revision: https://phab.mercurial-scm.org/D9113
mercurial/copies.py
--- a/mercurial/copies.py	Mon Sep 28 12:26:09 2020 +0200
+++ b/mercurial/copies.py	Fri Sep 25 14:33:05 2020 +0200
@@ -172,7 +172,7 @@
     return cm
 
 
-def _revinfogetter(repo):
+def _revinfo_getter(repo):
     """return a function that return multiple data given a <rev>"i
 
     * p1: revision number of first parent
@@ -278,7 +278,7 @@
 
     repo = a.repo().unfiltered()
     children = {}
-    revinfo = _revinfogetter(repo)
+    revinfo = _revinfo_getter(repo)
 
     cl = repo.changelog
     isancestor = cl.isancestorrev  # XXX we should had chaching to this.
@@ -309,12 +309,12 @@
     iterrevs.update(roots)
     iterrevs.remove(b.rev())
     revs = sorted(iterrevs)
-    return _combinechangesetcopies(
+    return _combine_changeset_copies(
         revs, children, b.rev(), revinfo, match, isancestor
     )
 
 
-def _combinechangesetcopies(
+def _combine_changeset_copies(
     revs, children, targetrev, revinfo, match, isancestor
 ):
     """combine the copies information for each item of iterrevs