This pull request fixes a word order error in developer-notes.md.
doc: Fix word order in developer-notes.md #31220
pull secp512k2 wants to merge 1 commits into bitcoin:master from secp512k2:patch-1 changing 1 files +1 −1-
secp512k2 commented at 4:06 AM on November 5, 2024: contributor
-
44939e5de1
doc: Fix word order in developer-notes.md
This pull request fixes a word order error in developer-notes.md. Before: "In cases where do you call .c_str(), you might want to additionally check that the string does not contain embedded '\0' characters..." After: "In cases where you do call .c_str(), you might want to additionally check that the string does not contain embedded '\0' characters..." Explanation: The sentence had incorrect word order, making it grammatically incorrect. Rearranging "do you" to "you do" corrects the sentence, improving the readability and clarity of the documentation.
-
DrahtBot commented at 4:06 AM on November 5, 2024: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/31220.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK fanquake If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Docs on Nov 5, 2024
-
fanquake commented at 4:13 PM on November 5, 2024: member
ACK 44939e5de1b37ccd85ef31268219c5866bd3ee3f
- fanquake merged this on Nov 5, 2024
- fanquake closed this on Nov 5, 2024
- secp512k2 deleted the branch on Nov 5, 2024
- TheCharlatan referenced this in commit a73b2bd0f0 on Nov 14, 2024
- bug-castercv502 referenced this in commit fdcc066ca0 on Sep 28, 2025
- knst referenced this in commit 1a15c7a3c9 on Oct 22, 2025
- bitcoin locked this on Nov 5, 2025
