Page MenuHomePhabricator
Differential D2739

[secp256k1] add ec_pubkey_parse to jni api
Needs RevisionPublic
Actions

Authored by floreslorca on Mar 26 2019, 02:25.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

implement parse pubkey

Test Plan

run testParsePubkey

Diff Detail

Repository
rABC Bitcoin ABC
Branch
parse
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5301
Build 8664: Bitcoin ABC Buildbot (legacy)
Build 8663: arc lint + arc unit

Event Timeline

floreslorca created this revision.Mar 26 2019, 02:25
Owners added a reviewer: Restricted Owners Package.Mar 26 2019, 02:25
Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 26 2019, 02:25
Herald added a reviewer: deadalnix. · View Herald Transcript
Herald added a subscriber: schancel. · View Herald Transcript
teamcity added a comment.Mar 26 2019, 02:26

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

teamcity added a comment.Mar 26 2019, 02:41

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B5301: Diff 7830.Mar 26 2019, 02:41
deadalnix requested changes to this revision.Mar 27 2019, 01:26
deadalnix added inline comments.
src/secp256k1/src/java/org/bitcoin/NativeSecp256k1.java
364

Why is that an ECDSA buffer?

src/secp256k1/src/java/org/bitcoin/NativeSecp256k1Test.java
179

There are no test cases for failures scenarii in parsePubKey

This revision now requires changes to proceed.Mar 27 2019, 01:26

Revision Contents
Changeset List

PathSize
src/
secp256k1/
src/
java/
org/
bitcoin/
37 lines
18 lines
41 lines

Diff 7830

View Options

src/secp256k1/src/java/org/bitcoin/NativeSecp256k1.java

Loading...
View Options

src/secp256k1/src/java/org/bitcoin/NativeSecp256k1Test.java

Loading...
View Options

src/secp256k1/src/java/org_bitcoin_NativeSecp256k1.c

Loading...