Page MenuHomePhabricator

Fix -Wthread-safety-analysis warnings
ClosedPublic

Authored by Fabien on Wed, Oct 23, 09:58.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCdf29943217b8: Fix -Wthread-safety-analysis warnings
Summary

The locks were missing from D4186.

Test Plan

With Clang as a compiler:

cmake -GNinja .. -DCMAKE_BUILD_TYPE=Debug
ninja check

Ensure there is no -Wthread-safety-analysis warning in the build
output, and all the tests are passing.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Wed, Oct 23, 09:58
Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, Oct 23, 09:58
deadalnix accepted this revision.Wed, Oct 23, 15:15
This revision is now accepted and ready to land.Wed, Oct 23, 15:15
This revision was automatically updated to reflect the committed changes.