diff options
author | MarcoFalke <falke.marco@gmail.com> | 2021-06-09 19:32:27 +0200 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2021-06-09 19:32:34 +0200 |
commit | ca424e242a718658785053217fd66bfb39b5e791 (patch) | |
tree | 4fca5dbecaa01d0145b0782aad5fd9c137644d4f /test/sanitizer_suppressions | |
parent | 93e38d5c06d9b32326585121b46bb59041d7cfed (diff) | |
parent | 3f05a9e681c4b72d99ddda8ccb6911d6ffad44ec (diff) | |
download | bitcoin-ca424e242a718658785053217fd66bfb39b5e791.tar.xz |
Merge bitcoin/bitcoin#22200: zmq: use std::string in zmqError()
3f05a9e681c4b72d99ddda8ccb6911d6ffad44ec zmq: use msg: prefix over errno= in zmqError (fanquake)
9a7cb57bbc61b2dfb772f8486db2a44c1673983a zmq: use std::string in zmqError() (fanquake)
Pull request description:
This is two minor changes. The first is to change `zmqError` to take a `const std::string&` instead of a `const char*`. The second is to change the second portion of `zmqError` to print `msg: message` rather than `errno=message`, given that `zmq_strerror` returns a message. To me, this seems more readable / useful than output like: `Error: Unable to initialize context errno=No such file or directory`.
ACKs for top commit:
practicalswift:
cr ACK 3f05a9e681c4b72d99ddda8ccb6911d6ffad44ec
instagibbs:
utACK https://github.com/bitcoin/bitcoin/pull/22200/commits/3f05a9e681c4b72d99ddda8ccb6911d6ffad44ec
theStack:
Code-Review ACK 3f05a9e681c4b72d99ddda8ccb6911d6ffad44ec
Tree-SHA512: 197cf381e8b3ced271d0e575e0c6d8e5e9ed93c4b284338b17873c5232eaabe64d6c4b66e1aeb5e76befc89e316abae2b28b7fd760f178481d7b9f4e3f85da67
Diffstat (limited to 'test/sanitizer_suppressions')
0 files changed, 0 insertions, 0 deletions