diff options
| author | MarcoFalke <[email protected]> | 2019-08-22 08:40:40 -0400 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2019-08-22 08:41:00 -0400 |
| commit | 52b9797119d5ee20b255efc68931ac7e158e325d (patch) | |
| tree | fe2d8a5f7c9e473c28ba0a086bafcaba8d6da25b /src/httpserver.h | |
| parent | Merge #16674: refactor: remove obsolete qt algorithm usage (diff) | |
| parent | rpc: Use Join helper in rpc/util (diff) | |
| download | discoin-52b9797119d5ee20b255efc68931ac7e158e325d.tar.xz discoin-52b9797119d5ee20b255efc68931ac7e158e325d.zip | |
Merge #16670: util: Add Join helper to join a list of strings
faebf6271467048dc8a9a0c526a0f8565023a966 rpc: Use Join helper in rpc/util (MarcoFalke)
fa8cd6f9c13319baca467864661982a3dfb2320c util: Add Join helper to join a list of strings (MarcoFalke)
Pull request description:
We have a lot of enumerations in the code and sometimes those enumerations need to be mentioned in the RPC or command line documentation. Previously, each caller would have a couple of lines inline to join the strings or the joined string is hardcoded in the documentation. A helper to join strings would make code such as https://github.com/bitcoin/bitcoin/pull/16629#discussion_r315852446 less verbose and easier to read.
Also, warnings commonly accumulate in complex RPCs, since a warning doesn't lead to an early return. A helper to join those warnings would make code such as https://github.com/bitcoin/bitcoin/pull/16394/files#r309324997 less verbose and easier to read.
ACKs for top commit:
practicalswift:
ACK faebf6271467048dc8a9a0c526a0f8565023a966
Tree-SHA512: 80f2db86a05c63b686f510585c1c631250271a8958fd71fafaac91559ffd2ec25d609bf7d53412ba27f87eff5893ac9dd9c2f296fc0c73581556e1d6a734a36f
Diffstat (limited to 'src/httpserver.h')
0 files changed, 0 insertions, 0 deletions