Page MenuHomePhabricator

[asmap] Remove SanityCheckASMap() from netaddress
ClosedPublic

Authored by PiRK on Jan 7 2022, 10:02.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC4efcc3345fac: [asmap] Remove SanityCheckASMap() from netaddress
Summary

SanityCheckASMap(asmap, bits) simply calls through to SanityCheckASMap(asmap)
in util/asmap. Update all callers to simply call that function.

This is a backport of core#22911 [2/4]
https://github.com/bitcoin/bitcoin/pull/22911/commits/bfdf4ef334a16ef6108a658bf4f8514754128c18#

Depends on D10779

Test Plan

ninja all check-all

Event Timeline

PiRK requested review of this revision.Jan 7 2022, 10:02

Tail of the build log:

[396/455] bitcoin: testing blockfilter_index_tests
[397/455] Running utility command for check-bitcoin-blockfilter_index_tests
[398/455] bitcoin: testing merkleblock_tests
[399/455] bitcoin: testing bip32_tests
[400/455] Running utility command for check-bitcoin-merkleblock_tests
[401/455] bitcoin: testing txvalidationcache_tests
[402/455] Running utility command for check-bitcoin-bip32_tests
[403/455] Running utility command for check-bitcoin-txvalidationcache_tests
[404/455] bitcoin: testing scheduler_tests
[405/455] bitcoin: testing sync_tests
[406/455] Running utility command for check-bitcoin-scheduler_tests
[407/455] bitcoin: testing cuckoocache_tests
[408/455] bitcoin: testing torcontrol_tests
[409/455] Running utility command for check-bitcoin-sync_tests
[410/455] Running utility command for check-bitcoin-cuckoocache_tests
[411/455] Running utility command for check-bitcoin-torcontrol_tests
[412/455] bitcoin: testing settings_tests
[413/455] Running utility command for check-bitcoin-settings_tests
[414/455] bitcoin: testing streams_tests
[415/455] bitcoin: testing timedata_tests
[416/455] Running utility command for check-bitcoin-streams_tests
[417/455] bitcoin: testing validation_flush_tests
[418/455] bitcoin: testing compilerbug_tests
[419/455] Running utility command for check-bitcoin-compilerbug_tests
[420/455] bitcoin: testing op_reversebytes_tests
[421/455] Running utility command for check-bitcoin-validation_flush_tests
[422/455] Running utility command for check-bitcoin-timedata_tests
[423/455] Running utility command for check-bitcoin-op_reversebytes_tests
[424/455] bitcoin: testing checkpoints_tests
[425/455] bitcoin: testing blockcheck_tests
[426/455] bitcoin: testing serialize_tests
[427/455] Running utility command for check-bitcoin-checkpoints_tests
[428/455] Running utility command for check-bitcoin-blockcheck_tests
[429/455] Running utility command for check-bitcoin-serialize_tests
[430/455] bitcoin: testing script_standard_tests
[431/455] bitcoin: testing validationinterface_tests
[432/455] Running utility command for check-bitcoin-script_standard_tests
[433/455] bitcoin: testing schnorr_tests
[434/455] Running utility command for check-bitcoin-validationinterface_tests
[435/455] Running utility command for check-bitcoin-schnorr_tests
[436/455] bitcoin: testing cashaddr_tests
[437/455] Running utility command for check-bitcoin-cashaddr_tests
[438/455] bitcoin: testing versionbits_tests
[439/455] Running utility command for check-bitcoin-versionbits_tests
[440/455] bitcoin: testing monolith_opcodes_tests
[441/455] Running utility command for check-bitcoin-monolith_opcodes_tests
[442/455] bitcoin: testing script_tests
[443/455] bitcoin: testing crypto_tests
[444/455] Running utility command for check-bitcoin-script_tests
[445/455] Running utility command for check-bitcoin-crypto_tests
[446/455] bitcoin: testing transaction_tests
[447/455] Running utility command for check-bitcoin-transaction_tests
[448/455] bitcoin: testing intmath_tests
[449/455] Running utility command for check-bitcoin-intmath_tests
[450/455] bitcoin: testing coinselector_tests
[451/455] Running utility command for check-bitcoin-coinselector_tests
[452/455] bitcoin: testing coins_tests
[453/455] Running utility command for check-bitcoin-coins_tests
ninja: build stopped: cannot make progress due to previous errors.
Build build-clang failed with exit code 1
This revision is now accepted and ready to land.Jan 7 2022, 11:05
This revision was landed with ongoing or failed builds.Jan 10 2022, 13:29
This revision was automatically updated to reflect the committed changes.