Page MenuHomePhabricator

Use TxId in policyestimator_tests.cpp
ClosedPublic

Authored by deadalnix on May 8 2018, 13:32.

Details

Summary

As per title, plus a few braces added for good measure.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
policyestimatortestnits
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2560
Build 3237: Bitcoin ABC Buildbot (legacy)
Build 3236: arc lint + arc unit

Event Timeline

jasonbcox requested changes to this revision.May 9 2018, 23:30
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
src/test/policyestimator_tests.cpp
72

Should the 10 here be changed to txIds.size() as well? If so, the comment below this needs to be updated as N/N blocks ..., N-1/N blocks ..., etc.

This revision now requires changes to proceed.May 9 2018, 23:30
src/test/policyestimator_tests.cpp
72

I think you are correct.

This revision is now accepted and ready to land.May 11 2018, 22:09
This revision was automatically updated to reflect the committed changes.