Feature/non inclusion #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

wanseob merged 3 commits into master from feature/non-inclusion

Nov 24, 2018

Contributor

wanseob commented Nov 24, 2018

From 1.1.0, we can also verify non inclusion

New functions

safeGet()
It guarantees the key-value existence
doesInclude()
You can check the key-value existence
getNonInclusionProof()
You can get data for non inclusion proof of a key
verifyNonInclusionProof()
You can check whether the key-value exists or not

wanseob added 3 commits

November 24, 2018 23:35


          Add safeGet feature for nonexistent key-value pair

c7913e0


          Add feature for non inclusion proof

1176eb8


          1.1.0

42d6f5a

coveralls commented Nov 24, 2018

Pull Request Test Coverage Report for Build 33

50 of 55 (90.91%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.8%) to 85.294%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
contracts/implementation.sol	4	5	80.0%
contracts/tree.sol	46	50	92.0%

Totals
Change from base Build 29:	0.8%
Covered Lines:	230
Relevant Lines:	248

💛 - Coveralls

wanseob merged commit 8eeebd1 into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet