Page MenuHomePhabricator
Differential D537

boost::shared_ptr => std::shared_ptr
ClosedPublic
Actions

Authored by deadalnix on Sep 8 2017, 14:33.

Details

Reviewers
freetrader
CCulianu
sickpig
Group Reviewers
Restricted Project
Commits
rSTAGINGeab65e757c61: boost::shared_ptr => std::shared_ptr
rABCeab65e757c61: boost::shared_ptr => std::shared_ptr
Summary

C++11 strikes again.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
stdshared_ptr
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 854
Build 854: arc lint + arc unit

Event Timeline

deadalnix created this revision.Sep 8 2017, 14:33
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 8 2017, 14:33
Harbormaster completed remote builds in B854: Diff 1360.Sep 8 2017, 14:33
CCulianu accepted this revision.Sep 10 2017, 18:59

I dream of the day when we don't need Boost for anything. Except maybe the test framework. If that.

Good job! Approved!

This revision is now accepted and ready to land.Sep 10 2017, 18:59
sickpig accepted this revision.Sep 11 2017, 08:53
Closed by commit rABCeab65e757c61: boost::shared_ptr => std::shared_ptr (authored by deadalnix, committed by deadalnix). · Explain WhySep 11 2017, 16:58
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
rpc/
8 lines
1 line
8 lines
wallet/
3 lines
4 lines

Diff 1360

View Options

src/rpc/mining.cpp

Loading...
View Options

src/rpc/server.cpp

Loading...
View Options

src/validationinterface.h

Loading...
View Options

src/wallet/wallet.h

Loading...
View Options

src/wallet/wallet.cpp

Loading...