( )⚙ D5513 amend: concise logic for rewrite.updatetimestamp

This is an archive of the discontinued Mercurial Phabricator instance.

amend: concise logic for rewrite.updatetimestamp
AbandonedPublic

Authored by taapas1128 on Jan 7 2019, 8:22 AM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Summary

This is a follow-up patch to D5491.

Logic for update time to current config option is made more concise as suggested by @pulkit .

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

taapas1128 created this revision.Jan 7 2019, 8:22 AM
taapas1128 edited the summary of this revision. (Show Details)Jan 7 2019, 8:25 AM
taapas1128 added a subscriber: pulkit.
pulkit added a comment.Jan 7 2019, 9:34 AM

Thanks for the quick follow-up. Yuya did it better and I pushed his patches which can be found at https://www.mercurial-scm.org/repo/hg-committed/rev/b153a4aa06f8 and https://www.mercurial-scm.org/repo/hg-committed/rev/cffa8e0ba77a. This one can be abandoned.

taapas1128 abandoned this revision.Jan 7 2019, 10:06 AM