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
