Differential D7657

Delete gej_is_valid_var: unused outside tests
ClosedPublic
Actions

Authored by deadalnix on Sep 29 2020, 12:44.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCaf04b3dd3be4: Delete gej_is_valid_var: unused outside tests

Summary

This is a partial backport of secp256k1 PR808 : https://github.com/bitcoin-core/secp256k1/pull/808/commits/d7f39ae4b67ea1ac6f085e6262a5f53afc0c5a25

Depends on D7656

Test Plan

ninja check-secp256k1

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

deadalnix created this revision.Sep 29 2020, 12:44

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 29 2020, 12:44

deadalnix requested review of this revision.Sep 29 2020, 12:44

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

deadalnix added a child revision: D7658: Make the curve B constant a secp256k1_fe.Sep 29 2020, 12:47

Fabien accepted this revision.Sep 29 2020, 12:59

This revision is now accepted and ready to land.Sep 29 2020, 12:59

This revision was landed with ongoing or failed builds.Sep 29 2020, 13:05

Closed by commit rABCaf04b3dd3be4: Delete gej_is_valid_var: unused outside tests (authored by Pieter Wuille <pieter@wuille.net>, committed by deadalnix). · Explain Why

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABCaf04b3dd3be4: Delete gej_is_valid_var: unused outside tests.

Harbormaster completed remote builds in B13039: Diff 23998.Sep 29 2020, 13:06

Revision Contents
Changeset List

Path

Size

src/

secp256k1/

src/

group_impl.h

20 lines

tests.c

2 lines

Diff 24015

View Options

src/secp256k1/src/group_impl.h