Paths

Table of Contentst

Differential D16161

test: refactor: clarify the coins simulation
ClosedPublic
Actions

Authored by PiRK on May 15 2024, 12:02.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABC44b0c160029a: test: refactor: clarify the coins simulation

Summary

Adds comments, slight refactor clarifications to make the code
easier to follow.

This is a partial backport of core#17487
https://github.com/bitcoin/bitcoin/pull/17487/commits/6d8affca96c7a34f5f104c5a3122e7420ffc083c

Depends on D16156

Test Plan

ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.May 15 2024, 12:02

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 15 2024, 12:02

PiRK requested review of this revision.May 15 2024, 12:02

teamcity edited the summary of this revision. (Show Details)May 15 2024, 12:03

PiRK added a child revision: D16162: coins: add Sync() method to allow flush without cacheCoins drop.May 15 2024, 12:05

Harbormaster completed remote builds in B29006: Diff 47779.May 15 2024, 13:01

Fabien accepted this revision.May 16 2024, 07:58

This revision is now accepted and ready to land.May 16 2024, 07:58

Closed by commit rABC44b0c160029a: test: refactor: clarify the coins simulation (authored by James O'Beirne <james.obeirne@gmail.com>, committed by PiRK). · Explain WhyMay 16 2024, 11:57

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC44b0c160029a: test: refactor: clarify the coins simulation.

Revision Contents
Changeset List

Path

Size

src/

test/

coins_tests.cpp

18 lines

Diff 47815

src/test/coins_tests.cpp

Loading...