When switching to a 256bit hash function, this still needs adjustment,
but concentrates the change in one place.
Details
Details
- Reviewers
pulkit - Group Reviewers
hg-reviewers - Commits
- rHG4f0e03d980f3: dirstate: isolate node len dependency for the pure version
Diff Detail
Diff Detail
- Repository
- rHG Mercurial
- Branch
- default
- Lint
No Linters Available - Unit
No Unit Test Coverage
Event Timeline
mercurial/dirstate.py | ||
---|---|---|
1670–1672 | I'm not convinced this change is valid. The heuristic is about dirstate entries which do not contain hashes, only a path and a few integers for size, mtime, state and mode. | |
1838 | I know this was already queued (published?), but this is part of the Rust impl and should not have been touched if the idea is only to fix the pure version, as this breaks the Rust version. |
mercurial/dirstate.py | ||
---|---|---|
1670–1672 | Hm. In that case the whole comment above it wrong. |
I'm not convinced this change is valid. The heuristic is about dirstate entries which do not contain hashes, only a path and a few integers for size, mtime, state and mode.