Remove the extremely outdated TODO file. #805

pull gmaxwell wants to merge 1 commits into bitcoin-core:master from gmaxwell:202008-notodo changing 1 files +0 −3

gmaxwell commented at 11:16 PM on August 31, 2020: contributor

This had two things in it-- tests for the scalar/field code and constant time signing and keygen.

The signing and keygen have been thoroughly constant time for years and there are now powerful tests to verify it... no further work on constant-time is needed at least on ordinary platforms (other sidechannels-- sure).

The scalar and field code have extensive tests. They could use better static test vectors but they're well tested.

TODOs for the project are currently better documented on github right now. This file could return in the future with current info, if needed.

Remove the extremely outdated TODO file.

This had two things in it-- tests for the scalar/field code and
 constant time signing and keygen.

The signing and keygen have been thoroughly constant time for years
 and there are now powerful tests to verify it...  no further work
 on constant-time is needed at least on ordinary platforms (other
 sidechannels-- sure).

The scalar and field code have extensive tests.  They could use
 better static test vectors but they're well tested.

TODOs for the project are currently better documented on github
 right now.  This file could return in the future with current
 info, if needed.

1c325199d5

real-or-random commented at 7:58 AM on September 1, 2020: contributor

ACK https://github.com/bitcoin-core/secp256k1/pull/805/commits/1c325199d590e018cdfb5ea2ab541774009bf7f7
sipa commented at 8:01 AM on September 1, 2020: contributor

Last updated in 2013 (7b03d836be9e3779dd23bb9e3447d51ecb27f89e)

ACK
real-or-random merged this on Sep 1, 2020
real-or-random closed this on Sep 1, 2020
gmaxwell deleted the branch on Sep 1, 2020
jasonbcox referenced this in commit 147e8bd648 on Sep 28, 2020
deadalnix referenced this in commit c29b8cdd16 on Sep 29, 2020

Contributors

gmaxwell

real-or-random

sipa