Page MenuHomePhabricator

Merge #15321: doc: Add cs_main lock annotations for mapBlockIndex
ClosedPublic

Authored by markblundeberg on Feb 10 2020, 12:45.

Details

Summary

PR15321 backport, aside from one minor change which will be noticed
when the backport in question happens later on ( PR14957 ).

Depends on D5237

Test Plan

ninja check all using clang compiler

Diff Detail

Repository
rABC Bitcoin ABC
Branch
mbicsmain
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9415
Build 16759: Default Diff Build & Tests
Build 16758: arc lint + arc unit

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those Bitcoin Core PRs have been inserted into the summary for reference.

deadalnix requested changes to this revision.Feb 10 2020, 14:15
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/wallet/test/wallet_tests.cpp
36 ↗(On Diff #16208)

Backporting in order would be much better. Alternatively, mentioning what actually needs to be seen and have a reference to the diff would be helpful rather than send the person who will backport on a wild chase.

This revision now requires changes to proceed.Feb 10 2020, 14:15
src/wallet/test/wallet_tests.cpp
36 ↗(On Diff #16208)

Yeah I looked into it, but those backports look a bit fragile to pull in just for this (the one in question is a bugfix and amusingly introduces arithmetic on null pointers -- https://github.com/bitcoin/bitcoin/pull/14957/files#diff-c0ec078346af03640b953e854ff6609aR344 -- which will be fixed eventually once T600 is done).

This revision is now accepted and ready to land.Feb 11 2020, 16:37