As discussed in #1453
This only changes the secp256k1_fe_impl_set_b32_mod comment since I think secp256k1_fe_set_b32_limit doc is clear enough.
doc: improve secp256k1_fe_set_b32_mod doc #1455
pull Coding-Enthusiast wants to merge 1 commits into bitcoin-core:master from Coding-Enthusiast:master changing 1 files +2 −1-
Coding-Enthusiast commented at 4:53 AM on December 8, 2023: contributor
- real-or-random approved
-
real-or-random commented at 9:35 AM on December 8, 2023: contributor
ACK e9c004ec532c19b5aaedfb62269ef060fc97fc11
Can you update the commit message to something more meaningful? E.g., just
doc: improve secp256k1_fe_set_b32_mod docas the PR title? - real-or-random added the label refactor/smell on Dec 8, 2023
-
doc: improve secp256k1_fe_set_b32_mod doc 3928b7c383
- Coding-Enthusiast force-pushed on Dec 8, 2023
- theStack approved
-
theStack commented at 11:45 PM on December 10, 2023: contributor
ACK 3928b7c38367947756b7d506f431bfb7bbbac5d0
-
sipa commented at 11:46 PM on December 10, 2023: contributor
ACK 3928b7c38367947756b7d506f431bfb7bbbac5d0
- real-or-random merged this on Dec 11, 2023
- real-or-random closed this on Dec 11, 2023
- fanquake referenced this in commit 41e1b677ca on Jan 3, 2024
- fanquake referenced this in commit 29fde0223a on Jan 4, 2024
- janus referenced this in commit 5fe435c9b2 on Apr 6, 2024
- hebasto referenced this in commit b6de625950 on May 11, 2024
- delta1 referenced this in commit 6089844b3c on Apr 2, 2025
- div72 referenced this in commit af627d47c3 on Apr 12, 2025
- str4d referenced this in commit 136aa9fe62 on Jun 4, 2025
- oskarszoon referenced this in commit 0d5c9260f4 on Jul 1, 2025