refactor: Use Mutex type for some mutexes in CNode class
ClosedPublic
Actions

Authored by PiRK on Apr 27 2021, 14:06.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rABC7ae0a8a18881: refactor: Use Mutex type for some mutexes in CNode class

Summary

No need the RecursiveMutex type for the CNode::cs_vSend, CNode::cs_hSocket and CNode::cs_vRecv.

This is a backport of core#19915

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr19915

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15589
Build 31094: Build Diff	lint-circular-dependencies · build-without-wallet · build-debug · build-diff · build-clang · build-clang-tidy
Build 31093: arc lint + arc unit

Event Timeline

PiRK created this revision.Apr 27 2021, 14:06

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 27 2021, 14:06

PiRK requested review of this revision.Apr 27 2021, 14:06

teamcity edited the summary of this revision. (Show Details)Apr 27 2021, 14:06

Harbormaster completed remote builds in B15589: Diff 28283.Apr 27 2021, 14:17

deadalnix accepted this revision.Apr 27 2021, 15:17

This revision is now accepted and ready to land.Apr 27 2021, 15:17

Closed by commit rABC7ae0a8a18881: refactor: Use Mutex type for some mutexes in CNode class (authored by Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>, committed by PiRK). · Explain WhyApr 27 2021, 17:38

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC7ae0a8a18881: refactor: Use Mutex type for some mutexes in CNode class.

Revision Contents
Changeset List

Path

Size

src/

net.h

6 lines

Diff 28283

View Options

refactor: Use Mutex type for some mutexes in CNode classClosedPublicActions