Skip to content

Commit ffc05fc

Browse files
committed
Merge bitcoin/bitcoin#31220: doc: Fix word order in developer-notes.md
44939e5 doc: Fix word order in developer-notes.md (secp512k2) Pull request description: This pull request fixes a word order error in developer-notes.md. ACKs for top commit: fanquake: ACK 44939e5 Tree-SHA512: f8c8f2a976940c6fb3483c13c048accd073a8486e0d614ca9da15bf166c41f1f1c1cd57678359af04a32c5ffa15afa647a3f015f89c83997c69803a62fad8de6
2 parents 9f2c828 + 44939e5 commit ffc05fc

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

doc/developer-notes.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -959,7 +959,7 @@ Strings and formatting
959959
- *Rationale*: Qt has built-in functionality for converting their string
960960
type from/to C++. No need to roll your own.
961961
962-
- In cases where do you call `.c_str()`, you might want to additionally check that the string does not contain embedded '\0' characters, because
962+
- In cases where you do call `.c_str()`, you might want to additionally check that the string does not contain embedded '\0' characters, because
963963
it will (necessarily) truncate the string. This might be used to hide parts of the string from logging or to circumvent
964964
checks. If a use of strings is sensitive to this, take care to check the string for embedded NULL characters first
965965
and reject it if there are any (see `ParsePrechecks` in `strencodings.cpp` for an example).

0 commit comments

Comments
 (0)