Page MenuHomePhabricator

[wip] Move primary globals to globals.h
AbandonedPublic

Authored by schancel on Nov 26 2018, 19:16.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
refactor-globals (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4126
Build 6323: Bitcoin ABC Buildbot (legacy)
Build 6322: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Nov 27 2018, 13:50
deadalnix added a subscriber: deadalnix.

I would advise strongly against moving these globals around unless you can ensure that initialization will be done in proper order. Initialization across modules is linker dependent, unless you want to specify it using a linker script so they'd better not depend on each other, which I wouldn't bet on.

src/Makefile.am
427

This being included in 3 libraries indicate that it's likely none of these 3 places is actually correct.

src/logging.cpp
27

Now you just lost any guarantees on initialization order.

src/qt/bitcoingui.cpp
1157

The semantic isn't the same anymore. In addition it doesn't look like any of the changes int hat file is related to the diff.

src/qt/coincontroldialog.cpp
18

Why twice ?

This revision now requires changes to proceed.Nov 27 2018, 13:50

Minor cleanup. Still doenst' link

schancel added inline comments.
src/Makefile.am
427

Indeed. I'm having trouble with getting it to link. Not sure why: https://reviews.bitcoinabc.org/P21