Page MenuHomePhabricator

Merge #10581: Simplify return values of GetCoin/HaveCoin(InCache)
ClosedPublic

Authored by nakihito on Tue, Jul 9, 21:07.

Details

Reviewers
deadalnix
Fabien
jasonbcox
markblundeberg
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCbd9e25638438: Merge #10581: Simplify return values of GetCoin/HaveCoin(InCache)
Summary

21180ff Simplify return values of GetCoin/HaveCoin(InCache) (Pieter Wuille)

Tree-SHA512: eae0aa64fa1308191100cdc7cdc790c825f33b066c200a18b5895d7d5806cee1cc4caba1766ef3379a7cf93dde4bbae2bc9be92947935f5741f5c126d3ee991b

Backport of Core PR10581
https://github.com/bitcoin/bitcoin/pull/10581

Test Plan
make check
test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nakihito created this revision.Tue, Jul 9, 21:07
Owners added a reviewer: Restricted Owners Package.Tue, Jul 9, 21:07
Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Jul 9, 21:07
deadalnix requested changes to this revision.Tue, Jul 9, 23:25
deadalnix added inline comments.
src/coins.h
141 ↗(On Diff #10160)

Fix style.

This revision now requires changes to proceed.Tue, Jul 9, 23:25
nakihito updated this revision to Diff 10165.Tue, Jul 9, 23:32

Fixed comment formatting.

deadalnix accepted this revision.Tue, Jul 9, 23:42
This revision is now accepted and ready to land.Tue, Jul 9, 23:42