Page MenuHomePhabricator

Add estimate size to dbwrapper
ClosedPublic

Authored by deadalnix on Aug 31 2017, 20:11.

Details

Summary

As per title.

Depends on D507

Test Plan
make check

D512 allow to test this better.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
dbwrapperestimate
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 843
Build 843: arc lint + arc unit

Event Timeline

schancel added inline comments.
src/dbwrapper.h
87

This would be easier to match with the comment if the constants weren't rolled together. However. Seems good.

104

I assume varint has some kind of variable encoding that indicates an extra byte?

src/dbwrapper.h
104

Correct.

This revision is now accepted and ready to land.Sep 6 2017, 16:35
This revision was automatically updated to reflect the committed changes.