Differential D12210

[GetTransaction] remove unneeded `cs_main` lock acquire
ClosedPublic
Actions

Authored by Fabien on Oct 11 2022, 18:32.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC2b1168a77909: [GetTransaction] remove unneeded `cs_main` lock acquire

Summary

Backport of core#22609.

Test Plan

With Debug and Clang:

ninja all check-all

Run the TSAN build.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR22609

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20445
Build 40562: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 40561: arc lint + arc unit

Event Timeline

Fabien created this revision.Oct 11 2022, 18:32

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 11 2022, 18:32

Fabien requested review of this revision.Oct 11 2022, 18:32

teamcity edited the summary of this revision. (Show Details)Oct 11 2022, 18:32

Harbormaster completed remote builds in B20445: Diff 35592.Oct 11 2022, 19:10

PiRK accepted this revision.Oct 12 2022, 04:22

This revision is now accepted and ready to land.Oct 12 2022, 04:22

Closed by commit rABC2b1168a77909: [GetTransaction] remove unneeded `cs_main` lock acquire (authored by Sebastian Falbesoner <sebastian.falbesoner@gmail.com>, committed by Fabien). · Explain WhyOct 12 2022, 07:49

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC2b1168a77909: [GetTransaction] remove unneeded `cs_main` lock acquire.

Revision Contents
Changeset List

Path

Size

src/

validation.cpp

2 lines

Diff 35592

View Options

[GetTransaction] remove unneeded `cs_main` lock acquireClosedPublicActions