Differential D7877

Add thread safety annotated wrapper for std::mutex
ClosedPublic
Actions

Authored by deadalnix on Oct 11 2020, 00:39.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC902d495d4e29: Add thread safety annotated wrapper for std::mutex

Summary

Co-authored-by: Anthony Towns <aj@erisian.com.au>

This is part [1/5] of Core PR18635 : https://github.com/bitcoin/bitcoin/pull/18635/commits/79be4874209f71ba6428a80c40c9f028ac936c41

Test Plan

ninja all check

Using clang. Check that the thread safety analysis is happy with it.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr18635-1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 13349
Build 26737: Build Diff	build-clang-tidy · build-without-wallet · lint-circular-dependencies · build-clang · build-diff
Build 26736: arc lint + arc unit

Event Timeline

deadalnix created this revision.Oct 11 2020, 00:39

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 11 2020, 00:39

deadalnix requested review of this revision.Oct 11 2020, 00:39

teamcity edited the summary of this revision. (Show Details)Oct 11 2020, 00:40

deadalnix added a child revision: D7878: refactor: Rename LockGuard to StdLockGuard for consistency with StdMutex.Oct 11 2020, 00:45

Harbormaster completed remote builds in B13349: Diff 24526.Oct 11 2020, 00:51

Fabien accepted this revision.Oct 11 2020, 11:18

This revision is now accepted and ready to land.Oct 11 2020, 11:18

Closed by commit rABC902d495d4e29: Add thread safety annotated wrapper for std::mutex (authored by Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>, committed by deadalnix). · Explain WhyOct 11 2020, 12:57

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC902d495d4e29: Add thread safety annotated wrapper for std::mutex.

Revision Contents
Changeset List

Path

Size

src/

logging.h

2 lines

threadsafety.h

10 lines

Diff 24526

View Options

src/logging.h