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 3278
Build 4642: Bitcoin ABC Buildbot (legacy)
Build 4641: 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

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 ↗(On Diff #4924)

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 ↗(On Diff #4924)

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