diff options
| author | fanquake <fanquake@gmail.com> | 2023-05-01 14:30:18 +0100 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2023-05-01 14:31:01 +0100 |
| commit | be0325c6a62505d63bc07320b05e31618ef9bbb1 (patch) | |
| tree | ec95b6322289742230dfda7e59db2e442e257803 /test/functional/test_framework/coverage.py | |
| parent | 539452242e895f2dcd719d41f447a48896d0e4b2 (diff) | |
| parent | dc14ba08e6e502f3e31d935bcd053a287c6610ca (diff) | |
Merge bitcoin/bitcoin#27538: test: Remove modinv python util helper function
dc14ba08e6e502f3e31d935bcd053a287c6610ca test: remove modinv python util helper function (Fabian Jahr)
Pull request description:
Since #27483 was merged the `modinv()` body is just one line calling pythons own implementation of `pow()`. We can just remove the function as it doesn't seem to add any value. Additionally the comment in the function is now outdated and the test is only testing two ways of doing modular inverse but both using python's `pow()` function.
ACKs for top commit:
theStack:
ACK dc14ba08e6e502f3e31d935bcd053a287c6610ca
Tree-SHA512: e8b470c72dc3f9fd53699d0684650517b1ea35ad1d4c01cf9472c80d3e4474c0c72e429c0bd201eb99d204c87eee0d68285e6a388e4c506f30e14b2bff9c1c32
Diffstat (limited to 'test/functional/test_framework/coverage.py')
0 files changed, 0 insertions, 0 deletions