mirror of
https://github.com/bitcoin/bitcoin.git
synced 2024-11-19 01:42:58 +01:00
Merge bitcoin/bitcoin#31220: doc: Fix word order in developer-notes.md
44939e5de1
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: ACK44939e5de1
Tree-SHA512: f8c8f2a976940c6fb3483c13c048accd073a8486e0d614ca9da15bf166c41f1f1c1cd57678359af04a32c5ffa15afa647a3f015f89c83997c69803a62fad8de6
This commit is contained in:
commit
ffc05fca6f
@ -959,7 +959,7 @@ Strings and formatting
|
||||
- *Rationale*: Qt has built-in functionality for converting their string
|
||||
type from/to C++. No need to roll your own.
|
||||
|
||||
- In cases where do you call `.c_str()`, you might want to additionally check that the string does not contain embedded '\0' characters, because
|
||||
- In cases where you do call `.c_str()`, you might want to additionally check that the string does not contain embedded '\0' characters, because
|
||||
it will (necessarily) truncate the string. This might be used to hide parts of the string from logging or to circumvent
|
||||
checks. If a use of strings is sensitive to this, take care to check the string for embedded NULL characters first
|
||||
and reject it if there are any (see `ParsePrechecks` in `strencodings.cpp` for an example).
|
||||
|
Loading…
Reference in New Issue
Block a user