[Updated] [+ ] D11691: dirstate: rename a `very_likely_equal` method to `likely_equal`

marmoute (Pierre-Yves David) phabricator at mercurial-scm.org
Tue Oct 19 22:03:01 UTC 2021


marmoute updated this revision to Diff 30914.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D11691?vs=30897&id=30914

BRANCH
  default

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D11691/new/

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

AFFECTED FILES
  rust/hg-core/src/dirstate/entry.rs
  rust/hg-core/src/dirstate_tree/status.rs

CHANGE DETAILS

diff --git a/rust/hg-core/src/dirstate_tree/status.rs b/rust/hg-core/src/dirstate_tree/status.rs
--- a/rust/hg-core/src/dirstate_tree/status.rs
+++ b/rust/hg-core/src/dirstate_tree/status.rs
@@ -200,7 +200,7 @@
                 // directory eligible for `read_dir` caching.
                 if let Some(meta) = directory_metadata {
                     if cached_mtime
-                        .very_likely_equal_to_mtime_of(meta)
+                        .likely_equal_to_mtime_of(meta)
                         .unwrap_or(false)
                     {
                         // The mtime of that directory has not changed
@@ -471,7 +471,7 @@
                     let is_up_to_date = if let Some(cached) =
                         dirstate_node.cached_directory_mtime()?
                     {
-                        cached.very_likely_equal(truncated)
+                        cached.likely_equal(truncated)
                     } else {
                         false
                     };
diff --git a/rust/hg-core/src/dirstate/entry.rs b/rust/hg-core/src/dirstate/entry.rs
--- a/rust/hg-core/src/dirstate/entry.rs
+++ b/rust/hg-core/src/dirstate/entry.rs
@@ -114,16 +114,16 @@
     /// If someone is manipulating the modification times of some files to
     /// intentionally make `hg status` return incorrect results, not truncating
     /// wouldn’t help much since they can set exactly the expected timestamp.
-    pub fn very_likely_equal(self, other: Self) -> bool {
+    pub fn likely_equal(self, other: Self) -> bool {
         self.truncated_seconds == other.truncated_seconds
             && self.nanoseconds == other.nanoseconds
     }
 
-    pub fn very_likely_equal_to_mtime_of(
+    pub fn likely_equal_to_mtime_of(
         self,
         metadata: &fs::Metadata,
     ) -> io::Result<bool> {
-        Ok(self.very_likely_equal(Self::for_mtime_of(metadata)?))
+        Ok(self.likely_equal(Self::for_mtime_of(metadata)?))
     }
 }
 



To: SimonSapin, #hg-reviewers, spectral
Cc: mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-patches/attachments/20211019/edf9faf7/attachment-0002.html>


More information about the Mercurial-patches mailing list