Patchwork D12514: dirstatemap: move `_drop_entry` out of the common methods

login
register
mail settings
Submitter phabricator
Date April 12, 2022, 4:03 p.m.
Message ID <differential-rev-PHID-DREV-g3d45m6culdt2wq3sea7-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/50862/
State New
Headers show

Comments

phabricator - April 12, 2022, 4:03 p.m.
Alphare created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  Only the Python implementation uses it.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D12514

AFFECTED FILES
  mercurial/dirstatemap.py

CHANGE DETAILS




To: Alphare, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

Patch

diff --git a/mercurial/dirstatemap.py b/mercurial/dirstatemap.py
--- a/mercurial/dirstatemap.py
+++ b/mercurial/dirstatemap.py
@@ -101,13 +101,6 @@ 
     def _refresh_entry(self, f, entry):
         """record updated state of an entry"""
 
-    def _drop_entry(self, f):
-        """remove any entry for file f
-
-        This should also drop associated copy information
-
-        The fact we actually need to drop it is the responsability of the caller"""
-
     ### disk interaction
 
     def _opendirstatefile(self):
@@ -534,6 +527,11 @@ 
             self._map.pop(f, None)
 
     def _drop_entry(self, f):
+        """remove any entry for file f
+
+        This should also drop associated copy information
+
+        The fact we actually need to drop it is the responsability of the caller"""
         self._map.pop(f, None)
         self.copymap.pop(f, None)