Page MenuHomePhabricator
Differential D9384

[SECP256K1] Remove unused secp256k1_fe_inv_all_var
ClosedPublic
Actions

Authored by Fabien on Apr 8 2021, 13:12.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCc47149e07ca0: [SECP256K1] Remove unused secp256k1_fe_inv_all_var
Summary
This function is only used in tests, since its functionality was inlined
into secp256k1_ge_set_all_gej_var in #553.

Backport of secp256k1#878

Test Plan
ninja check-secp256k1

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien created this revision.Apr 8 2021, 13:12
Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 8 2021, 13:12
Fabien requested review of this revision.Apr 8 2021, 13:12
teamcity edited the summary of this revision. (Show Details)Apr 8 2021, 13:12
deadalnix accepted this revision.Apr 8 2021, 13:20
This revision is now accepted and ready to land.Apr 8 2021, 13:20
This revision was landed with ongoing or failed builds.Apr 8 2021, 13:24
Closed by commit rABCc47149e07ca0: [SECP256K1] Remove unused secp256k1_fe_inv_all_var (authored by Pieter Wuille <pieter@wuille.net>, committed by Fabien). · Explain Why
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCc47149e07ca0: [SECP256K1] Remove unused secp256k1_fe_inv_all_var.
Harbormaster completed remote builds in B15467: Diff 28100.Apr 8 2021, 13:24

Revision Contents
Changeset List

PathSize
src/
secp256k1/
src/
5 lines
27 lines
52 lines

Diff 28111

View Options

src/secp256k1/src/field.h

Loading...
View Options

src/secp256k1/src/field_impl.h

Loading...
View Options

src/secp256k1/src/tests.c

Loading...