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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 ↗(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.