Page MenuHomePhabricator

Remove blockFinished from BlockAssembler

Authored by schancel on Jul 25 2018, 01:11.



As per title, stop using spooky action at a distance

Test Plan
make check && ./test/functional/

Diff Detail

rABC Bitcoin ABC
Lint Not Applicable
Tests Not Applicable

Event Timeline

Fix accidentally removed code during rebase

Remove a missed reference

This revision is now accepted and ready to land.Jul 25 2018, 02:03

Reword TestForBlockResult parameter to make it clearer

deadalnix requested changes to this revision.Jul 25 2018, 14:30
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
302 ↗(On Diff #4393)

You should move this in the header.

643 ↗(On Diff #4393)

This clearly changes the semantic. You tested for a specific package to be added, but then never adds it. Also, if blockFinished is true, the thing continue to loop until iterator runs out.

This revision now requires changes to proceed.Jul 25 2018, 14:30

Address some more feedback

deadalnix added inline comments.
128 ↗(On Diff #4402)

You can probably declare this as a private local class in BlockAssembler, as the only function using it is private anyway.

130 ↗(On Diff #4402)

TXCantFit ?

This revision is now accepted and ready to land.Jul 25 2018, 15:21
This revision was automatically updated to reflect the committed changes.