diff options
author | MarcoFalke <falke.marco@gmail.com> | 2019-08-22 08:40:40 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2019-08-22 08:41:00 -0400 |
commit | 52b9797119d5ee20b255efc68931ac7e158e325d (patch) | |
tree | fe2d8a5f7c9e473c28ba0a086bafcaba8d6da25b /doc | |
parent | 868a8cea15504118237da2f27bc8e79dd8af77f6 (diff) | |
parent | faebf6271467048dc8a9a0c526a0f8565023a966 (diff) | |
download | bitcoin-52b9797119d5ee20b255efc68931ac7e158e325d.tar.xz |
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 'doc')
0 files changed, 0 insertions, 0 deletions