This is an archive of the discontinued Mercurial Phabricator instance.

rust-dirstate: call new "dirs" rust implementation from Python
ClosedPublic

Authored by Alphare on May 17 2019, 6:10 AM.

Details

Summary

This is a simple module attribute replacement, will take precedence over the
Python and C implementations.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Alphare created this revision.May 17 2019, 6:10 AM
Alphare updated this revision to Diff 15343.Jun 5 2019, 12:23 PM
Alphare updated this revision to Diff 15509.Jun 14 2019, 6:14 AM

Updated to use the new Rust module policy

kevincox accepted this revision.Jun 14 2019, 6:18 AM
Alphare updated this revision to Diff 15672.Jun 27 2019, 9:17 AM
This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.
yuja added a subscriber: yuja.Jun 29 2019, 11:41 PM
  • a/rust/hg-core/src/dirstate/dirs_multiset.rs

+++ b/rust/hg-core/src/dirstate/dirs_multiset.rs
@@ -118,7 +118,9 @@

    entry.remove();
}
Entry::Vacant(_) => {
  • return Err(DirstateMapError::PathNotFound(path.to_owned()))

+ return Err(DirstateMapError::PathNotFound(
+ path.to_owned(),
+ ))

    }
};

Unrelated change. Dropped.