Page MenuHomePhabricator

Replace LockAssertion with a proper thread safety annotations
ClosedPublic

Authored by PiRK on Sep 21 2021, 12:12.

Details

Summary
Test Plan

With TSAN:
ninja all check-all

Event Timeline

PiRK requested review of this revision.Sep 21 2021, 12:12

remove accidental blank line

joeljporter retitled this revision from Replace LockAssertion with a proper thread safety annotations to Content Marketing Best Practices for 2021 .Sep 21 2021, 12:27
joeljporter edited the summary of this revision. (Show Details)
joeljporter edited the test plan for this revision. (Show Details)
joeljporter edited the summary of this revision. (Show Details)
joeljporter edited the test plan for this revision. (Show Details)
joeljporter edited the test plan for this revision. (Show Details)
PiRK retitled this revision from Content Marketing Best Practices for 2021 to Replace LockAssertion with a proper thread safety annotations.Sep 21 2021, 14:45
PiRK edited the summary of this revision. (Show Details)
PiRK edited the test plan for this revision. (Show Details)
Fabien added inline comments.
src/net_processing.cpp
1694 ↗(On Diff #30095)

Why is that causing an indentation change ? I think it should not

Apparently a linter issue

This revision is now accepted and ready to land.Sep 22 2021, 07:27
Unknown Object (User) added a subscriber: Unknown Object (User).Jul 21 2022, 04:43
This comment was removed by Mengerian.
Unknown Object (User) added a subscriber: Unknown Object (User).Mar 12 2024, 09:10
This comment was removed by Fabien.