Page MenuHomePhabricator

Backport all changes related to checkqueue.h and its test from Core
ClosedPublic

Authored by deadalnix on Sep 12 2018, 12:18.

Details

Summary

As per title. Backporting one by one did not seems realitic on that one.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
corepr9497
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3301
Build 4686: Bitcoin ABC Buildbot (legacy)
Build 4685: arc lint + arc unit

Event Timeline

schancel requested changes to this revision.Sep 15 2018, 15:11
schancel added a subscriber: schancel.
schancel added inline comments.
src/checkqueue.h
155 ↗(On Diff #4891)

Can you explain why this changed? Why swap instead of move?

This revision now requires changes to proceed.Sep 15 2018, 15:11
This revision is now accepted and ready to land.Sep 18 2018, 01:02
src/checkqueue.h
154

Seems you were the one responsible for changing this to std::move() in the first place in this commit:

https://reviews.bitcoinabc.org/D297

src/checkqueue.h
154

You are correct, std::move seems like a better option here.