Differential D9576

Add missing <optional> headers
ClosedPublic
Actions

Authored by Fabien on May 25 2021, 07:57.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCf7e0e1df575d: Add missing <optional> headers

Summary

This is a leftover from D8494: when switching from boost::optional to
std::optional the headers were not updated.

Also added a missing <limits> include in the FuzzDataProvider.h that
supposedly was pulled indirectly. Note that <climits> cannot be removed
due to CHAR_BIT.

Test Plan

On an up-to-date archlinux:

ninja all check
ninja bitcoin-fuzzers

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.May 25 2021, 07:57

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 25 2021, 07:57

Fabien requested review of this revision.May 25 2021, 07:57

Harbormaster completed remote builds in B15795: Diff 28589.May 25 2021, 08:15

PiRK accepted this revision.May 25 2021, 08:30

This revision is now accepted and ready to land.May 25 2021, 08:30

Closed by commit rABCf7e0e1df575d: Add missing <optional> headers (authored by Fabien). · Explain WhyMay 25 2021, 08:46

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCf7e0e1df575d: Add missing <optional> headers.

Revision Contents
Changeset List

Path

Size

src/

bench/

wallet_balance.cpp

7 lines

miner.h

1 line

node/

psbt.h

2 lines

psbt.h

2 lines

script/

descriptor.h

1 line

test/

fuzz/

1 line

1 line

1 line

1 line

util/

system.h

1 line

wallet/

2 lines

2 lines

2 lines

1 line

Diff 28591

View Options

src/bench/wallet_balance.cpp

View Options

src/miner.h

View Options

src/node/psbt.h

View Options

src/psbt.h

View Options

src/script/descriptor.h

View Options

src/test/fuzz/FuzzedDataProvider.h

View Options

src/test/fuzz/util.h

View Options

src/test/util_tests.cpp

View Options

src/txmempool.h

View Options

src/util/system.h

View Options

src/wallet/coincontrol.h

View Options

src/wallet/coinselection.cpp

View Options

src/wallet/rpcwallet.cpp

View Options

Add missing <optional> headersClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 28591

src/bench/wallet_balance.cpp

src/miner.h

src/node/psbt.h

src/psbt.h

src/script/descriptor.h

src/test/fuzz/FuzzedDataProvider.h

src/test/fuzz/util.h

src/test/util_tests.cpp

src/txmempool.h

src/util/system.h

src/wallet/coincontrol.h

src/wallet/coinselection.cpp

src/wallet/rpcwallet.cpp

src/wallet/wallet.h

Add missing <optional> headers
ClosedPublic
Actions

Revision Contents
Changeset List