This is an archive of the discontinued Mercurial Phabricator instance.

status: gather fixup info at comparison time

Authored by marmoute on Nov 24 2021, 6:11 AM.



This is still racy, but on a much small windows. In addition, the API now make
it possible for it to not be racy. This also unlock other cleanups that we are
about to do regarding mtime ambiguity at gathering time.

Diff Detail

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

Event Timeline

marmoute created this revision.Nov 24 2021, 6:11 AM
Alphare requested changes to this revision.Nov 24 2021, 8:22 AM
Alphare added a subscriber: Alphare.
Alphare added inline comments.



You accidentally a word


Maybe make the keyword explicit to be clearer? normal(f, parentfiledata=pfd)

This revision now requires changes to proceed.Nov 24 2021, 8:22 AM
marmoute added inline comments.Nov 30 2021, 12:25 PM

Actually we cannot pass keyword argument to lambda function.
(The argument becomes mandatory later anyway.)

marmoute edited the summary of this revision. (Show Details)Nov 30 2021, 6:53 PM
marmoute updated this revision to Diff 31226.

Sure, but one of the comments is still missing (at least) a word

marmoute updated this revision to Diff 31235.Dec 1 2021, 8:21 AM
Alphare accepted this revision.Dec 1 2021, 10:15 AM
This revision is now accepted and ready to land.Dec 1 2021, 10:15 AM
This revision was automatically updated to reflect the committed changes.