Differential D12581

node/caches: Remove intermediate variables
ClosedPublic
Actions

Authored by Fabien on Nov 22 2022, 16:01.

Details

Reviewers

sdulfari

Group Reviewers

Restricted Project

Commits

rABC19ca6c853413: node/caches: Remove intermediate variables

Summary

Partial backport of core#23280:
https://github.com/bitcoin/bitcoin/pull/23280/commits/ceb979034184345a662be4e3b327a573fbb31c63

Depends on D12580.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR23280_p18

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21054
Build 41762: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang · build-clang-tidy
Build 41761: arc lint + arc unit

Event Timeline

Fabien created this revision.Nov 22 2022, 16:01

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 22 2022, 16:01

Fabien requested review of this revision.Nov 22 2022, 16:01

teamcity edited the summary of this revision. (Show Details)Nov 22 2022, 16:02

Fabien added a child revision: D12582: node/chainstate: Add options for in-memory DBs.Nov 22 2022, 16:02

Harbormaster completed remote builds in B21054: Diff 36507.Nov 22 2022, 16:33

sdulfari accepted this revision.Nov 22 2022, 17:24

This revision is now accepted and ready to land.Nov 22 2022, 17:24

Closed by commit rABC19ca6c853413: node/caches: Remove intermediate variables (authored by Carl Dong <contact@carldong.me>, committed by Fabien). · Explain WhyNov 22 2022, 19:38

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC19ca6c853413: node/caches: Remove intermediate variables.

Revision Contents
Changeset List

Path

Size

src/

node/

caches.cpp

32 lines

Diff 36507

View Options

node/caches: Remove intermediate variablesClosedPublicActions