Add missing locks and locking annotations for CAddrMan
ClosedPublic
Actions

Authored by deadalnix on Jun 2 2019, 22:50.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING27cd5519cb9c: Add missing locks and locking annotations for CAddrMan
rABC27cd5519cb9c: Add missing locks and locking annotations for CAddrMan

Summary

This is a backport of Core PR13115

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr13115

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 6114
Build 10276: Bitcoin ABC Buildbot (legacy)
Build 10275: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jun 2 2019, 22:50

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 2 2019, 22:50

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B6114: Diff 9076.Jun 3 2019, 00:38

Fabien accepted this revision.Jun 3 2019, 06:41

This revision is now accepted and ready to land.Jun 3 2019, 06:41

Closed by commit rABC27cd5519cb9c: Add missing locks and locking annotations for CAddrMan (authored by practicalswift <practicalswift@users.noreply.github.com>, committed by deadalnix). · Explain WhyJun 3 2019, 12:34

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC27cd5519cb9c: Add missing locks and locking annotations for CAddrMan.

deadalnix added a commit: rSTAGING27cd5519cb9c: Add missing locks and locking annotations for CAddrMan.Sep 18 2019, 23:57

Revision Contents
Changeset List

Path

Size

src/

addrman.h

59 lines

test/

addrman_tests.cpp

8 lines

Diff 9076

View Options

src/addrman.h