( )⚙ D11518 dirstate-v2: initial Python parser

This is an archive of the discontinued Mercurial Phabricator instance.

dirstate-v2: initial Python parser
ClosedPublic

Authored by SimonSapin on Oct 1 2021, 3:18 AM.

Details

Summary

The dirstate-v2 file format should be supported even if Rust extensions are
not enabled. This changeset adds parsing code that is not used yet.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

SimonSapin created this revision.Oct 1 2021, 3:18 AM
marmoute requested changes to this revision.Oct 1 2021, 4:55 AM
marmoute added a subscriber: marmoute.

This change is actually quite simple to read. I could use a couple of doc-string though. Given the "unrelated" change issue to I am marking it as "Request-Changes", but the logic looks solid.

rust/hg-core/src/dirstate_tree/on_disk.rs
42–181

This change to the Rust code seems "unrelated"¹ and should go in a different patch, should they not ?

[1] i.e. they can be made independently on their own before this patch and still make sense.

This revision now requires changes to proceed.Oct 1 2021, 4:55 AM
marmoute retitled this revision from dirstate-v2: Initial Python parser to dirstate-v2: initial Python parser.
marmoute updated this revision to Diff 30499.
marmoute updated this revision to Diff 30615.Oct 3 2021, 7:45 AM
marmoute updated this revision to Diff 30620.Oct 3 2021, 7:51 AM
Alphare accepted this revision.Oct 3 2021, 5:22 PM
SimonSapin updated this revision to Diff 30689.Oct 8 2021, 12:01 PM
baymax updated this revision to Diff 30759.Oct 12 2021, 2:03 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

SimonSapin updated this revision to Diff 30774.Oct 13 2021, 1:24 PM
pulkit accepted this revision.Oct 13 2021, 3:16 PM
pulkit accepted this revision.
This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.