diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-06-01 08:59:15 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-06-01 09:01:29 +0200 |
| commit | 5cf92c32d1acc0e1a89df781118068ae8b5f4ded (patch) | |
| tree | d267d74d91995cfce79a96b9904a113a401b1b05 /src/net.cpp | |
| parent | 62d9ff9e6e1e136eef6f09f05e1d624f53641827 (diff) | |
| parent | ffff0d04425a616c14fc4a562e8ef93d286705f8 (diff) | |
| download | bitcoin-5cf92c32d1acc0e1a89df781118068ae8b5f4ded.tar.xz | |
Merge bitcoin/bitcoin#21969: refactor: Switch serialize to uint8_t (Bundle 1/2)
ffff0d04425a616c14fc4a562e8ef93d286705f8 refactor: Switch serialize to uint8_t (1/n) (MarcoFalke)
Pull request description:
Replace `char` -> `uint8_t` in serialization where a sign doesn't make sense (char might be signed/unsigned).
ACKs for top commit:
practicalswift:
cr ACK ffff0d04425a616c14fc4a562e8ef93d286705f8: patch looks correct and commit hash is ffffresh (was bbbbadass)
kristapsk:
ACK ffff0d04425a616c14fc4a562e8ef93d286705f8
Tree-SHA512: cda682280c21d37cc3a6abd62569732079b31d18df3f157aa28bed80bd6f9f29a7db5c133b1f57b3a8f8d5ba181a76e473763c6e26a2df6d9244813f56f893ee
Diffstat (limited to 'src/net.cpp')
| -rw-r--r-- | src/net.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/net.cpp b/src/net.cpp index dcb665354f..6f9f17ed4e 100644 --- a/src/net.cpp +++ b/src/net.cpp @@ -3048,7 +3048,7 @@ void CaptureMessage(const CAddress& addr, const std::string& msg_type, const Spa ser_writedata64(f, now.count()); f.write(msg_type.data(), msg_type.length()); for (auto i = msg_type.length(); i < CMessageHeader::COMMAND_SIZE; ++i) { - f << '\0'; + f << uint8_t{'\0'}; } uint32_t size = data.size(); ser_writedata32(f, size); |