upgrade: rename `_copyrevlogs` to `_clonerevlogs`
authorPierre-Yves David <pierre-yves.david@octobus.net>
Sat, 27 Jul 2019 19:25:47 +0200
changeset 42690 095dcdd0d55c
parent 42689 896fb9deeaf8
child 42691 5535a2201ff1
upgrade: rename `_copyrevlogs` to `_clonerevlogs` The underlying revlog method is named `clone`, keeping the naming consistent seems clearer. This is motivated to clarify the difference with an (upcoming) function that simply copy revlog files as is.
mercurial/upgrade.py
--- a/mercurial/upgrade.py	Sat Jul 27 19:58:17 2019 +0200
+++ b/mercurial/upgrade.py	Sat Jul 27 19:25:47 2019 +0200
@@ -533,7 +533,7 @@
         #reverse of "/".join(("data", path + ".i"))
         return filelog.filelog(repo.svfs, path[5:-2])
 
-def _copyrevlogs(ui, srcrepo, dstrepo, tr, deltareuse, forcedeltabothparents):
+def _clonerevlogs(ui, srcrepo, dstrepo, tr, deltareuse, forcedeltabothparents):
     """Copy revlogs between 2 repos."""
     revcount = 0
     srcsize = 0
@@ -745,7 +745,7 @@
         deltareuse = revlog.revlog.DELTAREUSEALWAYS
 
     with dstrepo.transaction('upgrade') as tr:
-        _copyrevlogs(ui, srcrepo, dstrepo, tr, deltareuse,
+        _clonerevlogs(ui, srcrepo, dstrepo, tr, deltareuse,
                      're-delta-multibase' in actions)
 
     # Now copy other files in the store directory.