Page MenuHomePhabricator

[secp256k1] add schnorr verify jni binding
ClosedPublic

Authored by floreslorca on Mar 27 2019, 22:00.

Details

Reviewers
deadalnix
jasonbcox
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGING6a1c4b55f411: [secp256k1] add schnorr verify jni binding
rABC6a1c4b55f411: [secp256k1] add schnorr verify jni binding
Summary

schnorr jni binding that will allow jvm support

Test Plan

Diff Detail

Repository
rABC Bitcoin ABC
Branch
sverify1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5325
Build 8712: Bitcoin ABC Buildbot (legacy)
Build 8711: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Mar 27 2019, 22:00
jasonbcox requested changes to this revision.Mar 27 2019, 23:25
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
src/secp256k1/src/java/org/bitcoin/NativeSecp256k1Test.java
185 ↗(On Diff #7847)

It would be less error prone and easier to read if these tests were formatted with an array of input strings and array of expected outputs. Then just loop over the arrays with a single assertEquals() call on each set.

src/secp256k1/src/java/org/bitcoin/Secp256k1Context.java
34 ↗(On Diff #7847)

Is this related to this diff?

This revision now requires changes to proceed.Mar 27 2019, 23:25

add requests and fix revisions

This revision is now accepted and ready to land.Mar 29 2019, 17:10
This revision was automatically updated to reflect the committed changes.