Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mockable clock type #25101

Merged
merged 1 commit into from
May 20, 2022
Merged

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented May 10, 2022

This will be used primarily by the addr time refactor (#24697) to make addr relay time type safe. However, it can also be used in other places, and can be reviewed independently, so I split it up.

@maflcko
Copy link
Member Author

maflcko commented May 10, 2022

Naming discussion welcome. Previous naming thread at #24697 (comment)

@DrahtBot
Copy link
Contributor

DrahtBot commented May 10, 2022

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

No conflicts as of last run.

@maflcko maflcko force-pushed the 2205-mock-clock- branch from fa66cf1 to fabee7a Compare May 11, 2022 14:44
src/util/time.h Outdated Show resolved Hide resolved
src/util/time.h Outdated Show resolved Hide resolved
@maflcko maflcko force-pushed the 2205-mock-clock- branch from fabee7a to fa201cc Compare May 11, 2022 18:03
src/util/time.h Outdated Show resolved Hide resolved
@maflcko maflcko force-pushed the 2205-mock-clock- branch from fa201cc to fa0f3ff Compare May 11, 2022 19:23
@maflcko
Copy link
Member Author

maflcko commented May 12, 2022

Rebased for changes in src/bitcoin-cli.cpp.

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Concept ACK / Edit: light code review, debug build, and tested running bitcoind locally on this branch

src/util/time.h Outdated Show resolved Hide resolved
Copy link
Contributor

@ajtowns ajtowns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

src/util/time.h Outdated Show resolved Hide resolved
Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK fa20781

Am running mainnet and signet nodes with this branch today.

src/util/time.h Show resolved Hide resolved
src/util/time.h Show resolved Hide resolved
@maflcko maflcko force-pushed the 2205-mock-clock- branch from fa20781 to fa305fd Compare May 18, 2022 16:58
@maflcko
Copy link
Member Author

maflcko commented May 18, 2022

Rebased. Should be trivial to re-review

@jonatack
Copy link
Member

ACK fa305fd per git range-diff 7b3343f fa20781 fa305fd

@ajtowns
Copy link
Contributor

ajtowns commented May 20, 2022

ACK fa305fd

@fanquake fanquake merged commit 6407c0e into bitcoin:master May 20, 2022
@maflcko maflcko deleted the 2205-mock-clock-😃 branch May 20, 2022 08:31
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 28, 2022
fa305fd Add mockable clock type and TicksSinceEpoch helper (MarcoFalke)

Pull request description:

  This will be used primarily by the addr time refactor (bitcoin#24697) to make addr relay time type safe. However, it can also be used in other places, and can be reviewed independently, so I split it up.

ACKs for top commit:
  jonatack:
    ACK fa305fd per `git range-diff 7b3343f fa20781 fa305fd`
  ajtowns:
    ACK fa305fd

Tree-SHA512: da00200126833c1f55b1b1e68f596eab5c9254e82b188ad17779c08ffd685e198a7c5270791b4b69a858dc6ba4e051fe0c8b445d203d356d0c884f6365ee1cfe
@bitcoin bitcoin locked and limited conversation to collaborators May 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants