Differential D11527

[avalanche] Use argsman instead of gArgs throughout processor
ClosedPublic
Actions

Authored by sdulfari on May 27 2022, 17:22.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC521a3758562b: [avalanche] Use argsman instead of gArgs throughout processor

Summary

This removes the global dependency on gArgs in the avalanche processor. argsman
is already available so there is no reason to not use it.

There is no change in behavior.

Test Plan

ninja check check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Branch

ava-prc-gargs

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19165
Build 38078: Build Diff	build-diff · lint-circular-dependencies · build-without-wallet · build-clang-tidy · build-debug · build-clang
Build 38077: arc lint + arc unit

Event Timeline

sdulfari created this revision.May 27 2022, 17:22

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 27 2022, 17:22

sdulfari requested review of this revision.May 27 2022, 17:22

Harbormaster completed remote builds in B19165: Diff 33707.May 27 2022, 17:34

Fabien accepted this revision.May 27 2022, 18:08

This revision is now accepted and ready to land.May 27 2022, 18:08

Closed by commit rABC521a3758562b: [avalanche] Use argsman instead of gArgs throughout processor (authored by sdulfari). · Explain WhyMay 30 2022, 22:36

This revision was automatically updated to reflect the committed changes.

sdulfari added a commit: rABC521a3758562b: [avalanche] Use argsman instead of gArgs throughout processor.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

processor.cpp

8 lines

Diff 33707

View Options

src/avalanche/processor.cpp

[avalanche] Use argsman instead of gArgs throughout processorClosedPublicActions