These abs checks are implied by the subsequent line, and with the subsequent line written as it is, no underflow is possible with signed integers.
Follows up on #1218.
Remove redundant checks. #1222
pull roconnor-blockstream wants to merge 1 commits into bitcoin-core:master from roconnor-blockstream:20230306_update_de changing 2 files +0 −4-
roconnor-blockstream commented at 2:16 PM on March 7, 2023: contributor
-
5d8f53e312
Remove redudent checks.
These abs checks are implied by the subsequent line, and with the subsequent line written as it is, no underflow is possible with signed integers.
- real-or-random approved
-
real-or-random commented at 2:20 PM on March 7, 2023: contributor
utACK 5d8f53e31293c582fb3fe02157bc67d2eeccea77
- sipa renamed this:
Remove redudent checks.
Remove redundant checks.
on Mar 7, 2023 -
sipa commented at 4:23 AM on March 8, 2023: contributor
utACK 5d8f53e31293c582fb3fe02157bc67d2eeccea77
- real-or-random approved
-
real-or-random commented at 11:26 AM on March 8, 2023: contributor
ACK 5d8f53e31293c582fb3fe02157bc67d2eeccea77
- real-or-random merged this on Mar 8, 2023
- real-or-random closed this on Mar 8, 2023
- sipa referenced this in commit 763079a3f1 on Mar 8, 2023
- div72 referenced this in commit 945b094575 on Mar 14, 2023
- vmta referenced this in commit e1120c94a1 on Jun 4, 2023
- vmta referenced this in commit 8f03457eed on Jul 1, 2023
- jonasnick cross-referenced this on Jul 20, 2023 from issue Upstream PRs 1160, 1193, 1169, 1190, 1192, 1194, 1196, 1195, 1170, 1172, 1200, 1199, 1203, 1201, 1206, 1078, 1209, 979, 1212, 1218, 1217, 1221, 1222 by jonasnick
Contributors